Skip to content

Address Book Edit Contact Hotfix#649

Merged
joemarct merged 1 commit intomasterfrom
hotfix/address-book
Mar 30, 2026
Merged

Address Book Edit Contact Hotfix#649
joemarct merged 1 commit intomasterfrom
hotfix/address-book

Conversation

@LukeHeimen
Copy link
Copy Markdown
Collaborator

@LukeHeimen LukeHeimen commented Mar 23, 2026

Description

Currently in the v0.24.0 (latest), when trying to update contact name or contact addresses, the edit dialog will not open. This is due to an error stating that the EditRecordDialog component is incorrectly defined, caused by a previous fix where the q-scanner and q-uploader components are moved outside the addresses-list component. This PR will attempt to fix the error by moving the q-scanner and q-uploader components inside the EditRecordDialog component.

Type of Change

  • UI improvements with no business logic changes
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Test Notes

The fix was tested locally and on an emulated Android device using Android Studio.

… component incorrectly defined; add setup function and emits declaration
@LukeHeimen LukeHeimen requested a review from joemarct March 23, 2026 02:15
@LukeHeimen LukeHeimen changed the title Edit Contact Hotfix Address Book Edit Contact Hotfix Mar 23, 2026
@opencode-agent
Copy link
Copy Markdown

Review Summary

This PR fixes a legitimate bug where the EditRecordDialog component was incorrectly structured. The changes are clean and correct:

What was broken:

  • QrScanner and QRUploader components were placed outside the q-dialog root element, violating Vue/Quasar's single-root requirement for dialog components
  • The dialog wasn't properly integrated with Quasar's Dialog Plugin API

What was fixed:

  1. Moved both scanner components inside the q-card within the dialog (lines 13-17)
  2. Added proper Quasar Dialog Plugin setup:
    • Added useDialogPluginComponent import
    • Declared proper emits with dialog events
    • Added setup() function returning dialogRef, onDialogHide, onDialogOK
    • Changed ref from editRecordDialogRef to dialogRef
    • Added @hide event handler
    • Replaced manual $emit('ok') and hide() calls with proper onDialogOK()

Verdict:Approve - This is a well-structured, minimal fix that follows Quasar's dialog plugin conventions properly.

New%20session%20-%202026-03-23T02%3A16%3A40.650Z
opencode session  |  github run

@joemarct joemarct merged commit ef9cb28 into master Mar 30, 2026
1 check passed
@joemarct joemarct deleted the hotfix/address-book branch March 30, 2026 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants