Skip to content

fix(collaborationAutocompletion): replace legacy SHARE_TYPE with ShareType#5419

Merged
hamza221 merged 1 commit into
mainfrom
fix/correct-sharetype-suggestion
Jul 1, 2026
Merged

fix(collaborationAutocompletion): replace legacy SHARE_TYPE with ShareType#5419
hamza221 merged 1 commit into
mainfrom
fix/correct-sharetype-suggestion

Conversation

@silverkszlo

Copy link
Copy Markdown
Contributor

Inside the Contacts app I could not add members to teams as no users could be found, although they existed.

Claude pointed me to that bug in collaborationAutocompletion.js where window.OC.Share.SHARE_TYPE_USER was still being used instead of ShareType.User and window.OC.Share.SHARE_TYPE_REMOTE instead of ShareType.Remote.

🤖 AI (if applicable)

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

… ShareType.User

Signed-off-by: silver <s.szmajduch@posteo.de>
@silverkszlo silverkszlo added bug Something isn't working 3. to review Waiting for reviews labels Jul 1, 2026
@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@hamza221

hamza221 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

/backport stable8.7

@hamza221 hamza221 merged commit 4ee707a into main Jul 1, 2026
35 checks passed
@welcome

welcome Bot commented Jul 1, 2026

Copy link
Copy Markdown

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/contacts/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@hamza221 hamza221 deleted the fix/correct-sharetype-suggestion branch July 1, 2026 13:39
@hamza221

hamza221 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

/backport to stable8.7 please

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 bug Something isn't working technical debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants