Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes reading on null error for mail notifications #15950

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Godmartinz
Copy link
Collaborator

Description

Checks to see if checkedOutTo is not null itself before failing on a locale check for mail notifications.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • [x ] Bug fix (non-breaking change which fixes an issue)

@Godmartinz Godmartinz requested a review from snipe as a code owner December 10, 2024 18:34
Copy link

what-the-diff bot commented Dec 10, 2024

PR Summary

  • Improved Safety Measures for null values
    The update ensures when accessing the locale attribute in the onCheckedOut and onCheckedIn methods, it won't cause a crash even if the value is not available (null). This makes the methods more reliable and less prone to errors in uncertain scenarios.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant