Skip to content

chore: load 'assign_default' value from legacy value #17428

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

Merged
merged 5 commits into from
Apr 16, 2025

Conversation

Emyrk
Copy link
Member

@Emyrk Emyrk commented Apr 16, 2025

If this value was set before v2.19.0, then assign_default was in a json field that would not match. And it would default to false. This corrects that.

If this value was set before v2.19.0, then assign_default was in
a json field that would not match. And it would default to `false`.
This corrects that.
@Emyrk Emyrk changed the title chore: 'assign_default' value should be loaded from legacy value chore: load 'assign_default' value from legacy value Apr 16, 2025
@Emyrk Emyrk requested a review from aslilac April 16, 2025 17:39
@stirby
Copy link
Collaborator

stirby commented Apr 16, 2025

/cherry-pick release/2.21


t.Run("AssignDefault,False", func(t *testing.T) {
t.Parallel()
legacy := `{
Copy link
Member

Choose a reason for hiding this comment

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

same here


t.Run("CorrectAssign", func(t *testing.T) {
t.Parallel()
legacy := `{
Copy link
Member

Choose a reason for hiding this comment

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

same here

Emyrk and others added 2 commits April 16, 2025 15:41
Co-authored-by: ケイラ <mckayla@hey.com>
@Emyrk Emyrk merged commit 2e5cd29 into main Apr 16, 2025
34 checks passed
@Emyrk Emyrk deleted the stevenmasley/org_sync_settings_fixed branch April 16, 2025 20:55
Copy link

Cherry-pick failed with Merge error 2e5cd299f2004a25e772c86c798a30df897a05b4 into temp-cherry-pick-93ca3c-release/2.21

@github-actions github-actions bot locked and limited conversation to collaborators Apr 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants