migrate ldap list wrapper fields#3162
Merged
DhariniJeeva merged 8 commits intollb/key-value-migrationfrom Feb 20, 2026
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Base automatically changed from
migrate-auth-methods-list-wrapper
to
llb/key-value-migration
February 10, 2026 22:25
DhariniJeeva
force-pushed
the
migrate-ldap-list-wrapper
branch
from
b31799b to
b20097a
Compare
DhariniJeeva
commented
Feb 10, 2026
ZedLi
previously approved these changes
Feb 11, 2026
Collaborator
ZedLi
left a comment
ZedLi
left a comment
There was a problem hiding this comment.
Looks good, just had one question
lisbet-alvarez
previously approved these changes
Feb 17, 2026
Collaborator
lisbet-alvarez
left a comment
lisbet-alvarez
left a comment
There was a problem hiding this comment.
Looks good! Thanks for the migration!
DhariniJeeva
dismissed stale reviews from ZedLi and lisbet-alvarez
The merge-base changed after approval.
DhariniJeeva
force-pushed
the
llb/key-value-migration
branch
from
e975866 to
17a1b55
Compare
chore: 🤖 name chore: 🤖 comments
chore: 🤖 remove comment chore: 🤖 remove unused chore: 🤖 account_attr_maps migration chore: 🤖 name fix: 🐛 name
DhariniJeeva
force-pushed
the
migrate-ldap-list-wrapper
branch
from
31786d2 to
a2e1208
Compare
Collaborator
Author
|
sorry for the noise but I'm closing and reopening this pr to check if that fixes the frozen state |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR updates the ldap listwrapper fields to keyValue
Screenshots (if appropriate)
certificates:
after change:
account_attribute_maps
before:
certificates:
account_attribute_maps
How to Test
Checklist
[ ] I have added steps to reproduce and test for bug fixes in the description[ ] I have addeda11y-testslabel to run a11y audit tests if neededPCI review checklist
Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.