Skip to content

[stable7.3] fix(deps): Upgrade @nextcloud/vue to fix focus-trap compatibility#4920

Merged
hamza221 merged 2 commits intostable7.3from
fix/focus-trap-version-mismatch
Feb 25, 2026
Merged

[stable7.3] fix(deps): Upgrade @nextcloud/vue to fix focus-trap compatibility#4920
hamza221 merged 2 commits intostable7.3from
fix/focus-trap-version-mismatch

Conversation

@nfebe
Copy link
Copy Markdown
Contributor

@nfebe nfebe commented Dec 4, 2025

Upgrade @nextcloud/vue from ^8.27.0 to ^8.34.0 to resolve focus-trap library version mismatch with the guests app. The mismatch caused "_setPausedState is not a function" error when opening the guest invite dialog from the Teams member picker.

Without this the "Invite guest" modal does not work over the "Add team members" modal.

@nfebe nfebe force-pushed the fix/focus-trap-version-mismatch branch from 31b818d to ff6515c Compare December 4, 2025 19:48
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@DaphneMuller
Copy link
Copy Markdown

@nfebe is it a problem that the tests are failing and who would be a good reviewer for this?
@janepie will also give a review

@SebastianKrupinski
Copy link
Copy Markdown
Contributor

Hi @nfebe

This probably needs a transformation exclusion like in calendar.

https://github.com/nextcloud/calendar/blob/5f1e875a3d36291bbd394c628efdb0e0c97d8c88/package.json#L150

@nfebe
Copy link
Copy Markdown
Contributor Author

nfebe commented Dec 8, 2025

@nfebe is it a problem that the tests are failing and who would be a good reviewer for this?

Yes, I made a fix but I will review the recently suggested workaround as well.

Should be done today.

1 similar comment
@nfebe
Copy link
Copy Markdown
Contributor Author

nfebe commented Dec 8, 2025

@nfebe is it a problem that the tests are failing and who would be a good reviewer for this?

Yes, I made a fix but I will review the recently suggested workaround as well.

Should be done today.

Comment thread tests/setup.js
@janepie janepie self-requested a review December 12, 2025 14:46
Copy link
Copy Markdown
Member

@janepie janepie left a comment

Choose a reason for hiding this comment

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

Fixes the issue

@SebastianKrupinski
Copy link
Copy Markdown
Contributor

@nfebe we still merging this?

@hamza221
Copy link
Copy Markdown
Contributor

@nfebe I'm closing this as 7.3 hit eol, lmk if you think it's still needed

@hamza221 hamza221 closed this Feb 25, 2026
@hamza221 hamza221 changed the title fix(deps): Upgrade @nextcloud/vue to fix focus-trap compatibility [stable7.3] fix(deps): Upgrade @nextcloud/vue to fix focus-trap compatibility Feb 25, 2026
@hamza221 hamza221 reopened this Feb 25, 2026
Upgrade @nextcloud/vue from ^8.27.0 to ^8.34.0 to resolve focus-trap
library version mismatch with the guests app. The mismatch caused
"_setPausedState is not a function" error when opening the guest
invite dialog from the Teams member picker.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
The jsdom test environment does not provide structuredClone, which is
used by @nextcloud/vue components.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@hamza221 hamza221 force-pushed the fix/focus-trap-version-mismatch branch from bdc576e to acb4df2 Compare February 25, 2026 15:55
@hamza221 hamza221 enabled auto-merge February 25, 2026 15:59
@hamza221 hamza221 added bug Something isn't working 4. to release Ready to be released and/or waiting for tests to finish feature: teams ex circles labels Feb 25, 2026
@hamza221 hamza221 merged commit e61e6ee into stable7.3 Feb 25, 2026
42 checks passed
@hamza221 hamza221 deleted the fix/focus-trap-version-mismatch branch February 25, 2026 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working feature: teams ex circles

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants