Skip to content

fix: addressbook checkbox - #5688

Merged
GVodyanov merged 1 commit into
mainfrom
fix/addressbook-checkbox
Sep 3, 2026
Merged

fix: addressbook checkbox#5688
GVodyanov merged 1 commit into
mainfrom
fix/addressbook-checkbox

Conversation

@GVodyanov

@GVodyanov GVodyanov commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Fixes this:

Screencast.From.2026-08-30.12-19-13.mp4

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Grigory Vodyanov <scratchx@gmx.com>
@GVodyanov GVodyanov self-assigned this Aug 29, 2026
@GVodyanov GVodyanov added the 2. developing Work in progress label Aug 29, 2026
@codecov

codecov Bot commented Aug 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@GVodyanov GVodyanov added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 30, 2026
@GVodyanov
GVodyanov marked this pull request as ready for review August 30, 2026 10:22
Comment thread src/store/addressbooks.js
@GVodyanov

GVodyanov commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

Note: this only happens for addressbooks that were just created, forgot to mention

@GVodyanov
GVodyanov requested a review from odzhychko September 3, 2026 12:10
Comment thread src/store/addressbooks.js
renameAddressbook(context, { addressbook, newName }) {
addressbook = state.addressbooks.find((search) => search.id === addressbook.id)
addressbook.displayName = newName
renameAddressbook(state, { addressbook, newName }) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Intentional?

@odzhychko odzhychko left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Note: this only happens for addressbooks that were just created, forgot to mention

Could reproduce now and the fix works.

@GVodyanov
GVodyanov merged commit 015fad2 into main Sep 3, 2026
46 checks passed
@GVodyanov
GVodyanov deleted the fix/addressbook-checkbox branch September 3, 2026 12:36
@GVodyanov

Copy link
Copy Markdown
Contributor Author

/backport to stable8.8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants