Skip to content

feat(config): add serverid configuration and use it for Snowflakes #56095

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.

Sign up for GitHub

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

Jump to bottom
Merged
Altahrim merged 1 commit into from
Nov 4, 2025
Merged

feat(config): add serverid configuration and use it for Snowflakes #56095

Altahrim merged 1 commit into from
Nov 4, 2025

Conversation

@Altahrim
Copy link
Collaborator

@Altahrim Altahrim commented Oct 30, 2025 •
edited
Loading

Summary

Introduce serverid to uniquely identify server.
Use it fin Snowflake IDs generator.

Checklist

@Altahrim Altahrim added this to the Nextcloud 33 milestone Oct 30, 2025
@Altahrim Altahrim self-assigned this Oct 30, 2025
@Altahrim Altahrim added 2. developing Work in progress php Pull requests that update Php code pending documentation This pull request needs an associated documentation update labels Oct 30, 2025
@github-project-automation github-project-automation bot moved this to 🏗️ In progress in 📁 Files team Oct 30, 2025
@Altahrim Altahrim force-pushed the feat/add-serverid branch 2 times, most recently from d05282d to 9d30c67 Compare October 31, 2025 08:39
@Altahrim Altahrim added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Oct 31, 2025
@Altahrim Altahrim marked this pull request as ready for review October 31, 2025 08:39
@Altahrim Altahrim requested a review from a team as a code owner October 31, 2025 08:39
@Altahrim Altahrim requested review from icewind1991 and salmart-dev and removed request for a team October 31, 2025 08:40
artonge
artonge approved these changes Oct 31, 2025
View reviewed changes
@Altahrim Altahrim force-pushed the feat/add-serverid branch 2 times, most recently from 13de616 to 20a1b60 Compare November 3, 2025 08:44
@Altahrim Altahrim mentioned this pull request Nov 3, 2025
Merged
@Altahrim
Copy link
Collaborator Author

Altahrim commented Nov 3, 2025

@Altahrim Altahrim removed the pending documentation This pull request needs an associated documentation update label Nov 3, 2025
@Altahrim Altahrim enabled auto-merge November 3, 2025 11:36
@Altahrim Altahrim merged commit 635a6ac into master Nov 4, 2025
190 checks passed
@Altahrim Altahrim deleted the feat/add-serverid branch November 4, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@artonge artonge artonge approved these changes

@CarlSchwan CarlSchwan CarlSchwan approved these changes

@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 is a code owner automatically assigned from nextcloud/server-backend

@salmart-dev salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend

Assignees

@Altahrim Altahrim

Labels

3. to review Waiting for reviews enhancement php Pull requests that update Php code

Projects

📁 Files team
Status: 🏗️ In progress

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

4 participants

@Altahrim @artonge @CarlSchwan