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

Release 6.8.3 - Ignore upper/lowercase differences in email when syncing external groups #371

Merged
merged 2 commits into from
Sep 18, 2024

Conversation

forevermatt
Copy link
Contributor

IDP-1153 REAP & Preservica's IDP customization for permission groups (via SSO)


Fixed

  • Ignore upper/lowercase differences in email when syncing external groups
    • Whether the Google Sheet has the non-lowercase email address or the ID Broker database has the non-lowercase email, this will now correctly sync the external groups (rather than removing them on every other run).
  • Update dependencies

Feature PR Checklist

  • Tests created or updated
  • Run make composershow
  • Run make psr2

Whether the Google Sheet has the non-lowercase email address or the ID
Broker database has the non-lowercase email, this will now correctly
sync the external groups (rather than removing them on every other run).
@forevermatt forevermatt requested a review from a team as a code owner September 18, 2024 18:06
@forevermatt forevermatt requested review from briskt, mtompset, jason-jackson and hobbitronics and removed request for a team September 18, 2024 18:06
Copy link

@forevermatt forevermatt merged commit ca81524 into main Sep 18, 2024
3 checks passed
@forevermatt forevermatt deleted the feature/IDP-1153-fix-capitalization-bug-in-sync branch September 18, 2024 20:15
@forevermatt forevermatt changed the title Ignore upper/lowercase differences in email when syncing external groups Release 6.8.3 - Ignore upper/lowercase differences in email when syncing external groups Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants