Skip to content

Bug fix LG-15274 add user to socure user set change define method#11828

Merged
AShukla-GSA merged 4 commits intomainfrom
socure-defined-bug-fix-2
Jan 30, 2025
Merged

Bug fix LG-15274 add user to socure user set change define method#11828
AShukla-GSA merged 4 commits intomainfrom
socure-defined-bug-fix-2

Conversation

@AShukla-GSA
Copy link
Copy Markdown
Contributor

🎫 Ticket

Bug Fix:
Link to the relevant ticket:
LG-15274

🛠 Summary of changes

Changing is_defined? to defined?

@AShukla-GSA
Copy link
Copy Markdown
Contributor Author

To track regression test: https://cm-jira.usa.gov/browse/LG-15642

Copy link
Copy Markdown
Contributor

@amirbey amirbey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AShukla-GSA AShukla-GSA merged commit 2039d35 into main Jan 30, 2025
@AShukla-GSA AShukla-GSA deleted the socure-defined-bug-fix-2 branch January 30, 2025 22:03
matthinz pushed a commit that referenced this pull request Jan 30, 2025
…1828)

* changelog: Bug Fixes, socure, changing is_defined to defined method

* including hybrid concer and adding tests

* adding safe check and raising error if uuid is nil in add user

* reverting raising error and removing hybrid concern from doc auth vendor concern
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.

3 participants