Skip to content

docs: add preference migration guide #10326

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.

Sign up for GitHub

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

Jump to bottom
Merged
wmontwe merged 1 commit into from
Jan 7, 2026
Merged

docs: add preference migration guide #10326

wmontwe merged 1 commit into from
Jan 7, 2026
+178 −0

Conversation

@wmontwe
Copy link
Member

@wmontwe wmontwe commented Jan 7, 2026

Resolves #10325

@wmontwe wmontwe requested a review from a team as a code owner January 7, 2026 13:13
@wmontwe wmontwe requested a review from rafaeltonholo January 7, 2026 13:13
@wmontwe wmontwe mentioned this pull request Jan 7, 2026
Merged
rafaeltonholo
rafaeltonholo reviewed Jan 7, 2026
View reviewed changes
Copy link
Member

@rafaeltonholo rafaeltonholo left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I would also include a section talking about writing unit tests for the Migration, and set it as a requirement instead of just mentioning it as a Best practice.

@wmontwe wmontwe force-pushed the docs-preference-migration branch 2 times, most recently from db8583d to 09e2e82 Compare January 7, 2026 13:48
@wmontwe wmontwe requested a review from rafaeltonholo January 7, 2026 13:48
rafaeltonholo
rafaeltonholo previously approved these changes Jan 7, 2026
View reviewed changes
Copy link
Member

@rafaeltonholo rafaeltonholo left a comment

Choose a reason for hiding this comment

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

👍

@wmontwe wmontwe force-pushed the docs-preference-migration branch from 09e2e82 to 930dd64 Compare January 7, 2026 13:52
@wmontwe
Copy link
Member Author

wmontwe commented Jan 7, 2026

Sorry updated again, forgot to push the test section.

@wmontwe wmontwe requested a review from rafaeltonholo January 7, 2026 13:52
rafaeltonholo
rafaeltonholo approved these changes Jan 7, 2026
View reviewed changes
Copy link
Member

@rafaeltonholo rafaeltonholo left a comment

Choose a reason for hiding this comment

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

Nice, thanks for adding the suggestions!

@wmontwe wmontwe merged commit c4cc69b into main Jan 7, 2026
5 checks passed
@wmontwe wmontwe deleted the docs-preference-migration branch January 7, 2026 14:04
@thunderbird-botmobile thunderbird-botmobile bot added this to the Thunderbird 17 milestone Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rafaeltonholo rafaeltonholo rafaeltonholo approved these changes

Assignees

@rafaeltonholo rafaeltonholo

Labels

None yet

Projects

None yet

Milestone

Thunderbird 17

Development

Successfully merging this pull request may close these issues.

Document the preference migration

3 participants

@wmontwe @rafaeltonholo