Skip to content

Display backups on dashboard - #2846

Merged
dwwoelfel merged 1 commit into
mainfrom
backups-flag
Aug 7, 2026
Merged

Display backups on dashboard#2846
dwwoelfel merged 1 commit into
mainfrom
backups-flag

Conversation

@dwwoelfel

@dwwoelfel dwwoelfel commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Enables the backups page on the dashboard for everyone.

I confirmed in production that we could both create backups on-demand and download backups.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The change removes the backups feature flag and its dashboard usage. The Backups tab is now filtered by role only.

Changes

Backups flag removal

Layer / File(s) Summary
Remove backups flag control
client/www/lib/flags.ts, client/www/pages/dash/index.tsx
The exported flags omit backups. The dashboard removes useFlag and showBackups, and filters tabs by role only.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly states that the pull request displays backups on the dashboard.
Description check ✅ Passed The description explains that the backups page is enabled for everyone and confirms backup creation and download behavior.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

View Vercel preview at instant-www-js-backups-flag-jsv.vercel.app.

@nezaj nezaj left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Woohoo!

@dwwoelfel
dwwoelfel merged commit d37218a into main Aug 7, 2026
29 checks passed
@dwwoelfel
dwwoelfel deleted the backups-flag branch August 7, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants