Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Mar 10, 2025

This PR consists of 3 commits:

  1. Switch upload data for lookup server to default false
  2. Switch query lookup server to default false
  3. Disable lookup server functionality if not on global scale

@susnux susnux added bug 3. to review Waiting for reviews labels Mar 10, 2025
@susnux susnux changed the title Fix/lookup server connector v2 fix(lookup-server): disable when not using global scale Mar 10, 2025
@susnux susnux marked this pull request as ready for review March 10, 2025 14:58
@susnux susnux requested review from a team as code owners March 10, 2025 14:58
@susnux susnux requested review from Pytal, artonge, sorbaugh and yemkareems and removed request for a team March 10, 2025 14:58
@susnux susnux force-pushed the fix/lookup-server-connector-v2 branch 2 times, most recently from 4d95896 to 172b661 Compare March 10, 2025 15:17
@susnux susnux force-pushed the fix/lookup-server-connector-v2 branch from 172b661 to 8787c75 Compare March 10, 2025 15:24
@susnux susnux requested a review from nickvergessen March 10, 2025 15:26
@susnux susnux force-pushed the fix/lookup-server-connector-v2 branch from 8787c75 to d899b57 Compare March 10, 2025 15:57
Copy link
Member

@ArtificialOwl ArtificialOwl left a comment

Choose a reason for hiding this comment

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

I would have preferred a single check on 'gs.enable' while commenting the original condition

@susnux susnux force-pushed the fix/lookup-server-connector-v2 branch from d899b57 to 7ab38b3 Compare March 11, 2025 08:26
@ArtificialOwl
Copy link
Member

/backport to stable31

@susnux susnux force-pushed the fix/lookup-server-connector-v2 branch from 7ab38b3 to 86767fb Compare March 11, 2025 13:26
@susnux susnux merged commit 38e8f30 into master Mar 11, 2025
190 checks passed
@susnux susnux deleted the fix/lookup-server-connector-v2 branch March 11, 2025 20:17
@AndyScherzinger AndyScherzinger added this to the Nextcloud 32 milestone Mar 11, 2025
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 19, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants