Skip to content
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

fix: use api to read input value #8021

Merged
merged 1 commit into from
Feb 15, 2023
Merged

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented Feb 14, 2023

The internal dom structure was changed by nextcloud-libraries/nextcloud-vue#3517

stable2.2 uses nextcloud/vue 7.1 => no backport

Test Case 1:

  • Open menu for account in navigation
  • Click "Add mailbox"
  • Add a mailbox

Test Case 2:

  • Open menu for mailbox in navigation
  • Click "Add submailbox"
  • Add a sub mailbox

It does not work on main because the internal dom strucure was changed.

@kesselb kesselb self-assigned this Feb 14, 2023
@kesselb kesselb added this to the v2.3.0 milestone Feb 14, 2023
The internal dom structure was changed by nextcloud-libraries/nextcloud-vue#3517

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
Copy link
Contributor

@GretaD GretaD left a comment

Choose a reason for hiding this comment

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

👍

@kesselb kesselb force-pushed the bug-add-mailbox-broken branch 2 times, most recently from 3ee7a8e to 89b3f9c Compare February 14, 2023 17:33
@ChristophWurst ChristophWurst merged commit db682f5 into main Feb 15, 2023
@ChristophWurst ChristophWurst deleted the bug-add-mailbox-broken branch February 15, 2023 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

3 participants