Skip to content

Comments

feat(frontend): add backup and redeploy app action with auth dialog#297

Open
drik98 wants to merge 13 commits intoaixigo:masterfrom
drik98:feature/improve-backup-ux
Open

feat(frontend): add backup and redeploy app action with auth dialog#297
drik98 wants to merge 13 commits intoaixigo:masterfrom
drik98:feature/improve-backup-ux

Conversation

@drik98
Copy link
Contributor

@drik98 drik98 commented Feb 19, 2026

Summary

This PR improves backup UX by adding an app-level Back up / Redeploy action directly in each app card menu, so users no longer need to call the REST API manually.

What Changed

  • Added a new dropdown action in ReviewAppCard:
    • Shows Back up for deployed apps
    • Shows Redeploy for backed-up apps
  • Added new confirmation modal: BackupAppDialog.vue
    • Same confirmation pattern as shutdown
    • Auth-aware warning: You need to be logged in to back up or redeploy apps.
    • Refactored existing dialogs to de-duplicate logic
  • Added Vuex app-state toggle support in frontend/src/store/index.js:
    • New action: changeAppState
    • Calls PUT /api/apps/{appName}/states/
    • Reuses fetchAndPoll for async task completion handling
  • Backed-up apps are now displayed under the "My Previews" section if the owner matches
    • Backed-up apps now have a badge in the heading indicating they are backed-up to distinguish them better
  • Improved the logic around expansion of container of a review app
    • Added unit tests to document the intentions better

@drik98 drik98 force-pushed the feature/improve-backup-ux branch from f983226 to ff684c3 Compare February 19, 2026 08:26
@drik98 drik98 changed the base branch from master to automatic-back-up February 19, 2026 08:26
@drik98 drik98 marked this pull request as ready for review February 19, 2026 08:45
Copy link
Contributor

@schrieveslaach schrieveslaach left a comment

Choose a reason for hiding this comment

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

Thanks for stepping in here. I was planning to work this today or tomorrow. Looks very promising to me. However, there is one thing I like too ask you: can you check if there is a nice way of making the dashboard a bit more compact by moving the backups associated with the logged in user into the same slot “My Previews”?

@drik98
Copy link
Contributor Author

drik98 commented Feb 20, 2026

@schrieveslaach I think we will also have to remove the Logs link of the service of an back-upped app right? Currently we can still open the logs window but the API returns 400.

Copy link
Contributor

@schrieveslaach schrieveslaach left a comment

Choose a reason for hiding this comment

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

Thanks for putting in the effort for making the dashboard more compact and for fixing the input focus issue. I really appreciate.

The only missing part is the isBackupsEnabled condition.

@schrieveslaach
Copy link
Contributor

@schrieveslaach I think we will also have to remove the Logs link of the service of an back-upped app right? Currently we can still open the logs window but the API returns 400.

If you are so kind, pleas, go ahead. Sounds reasonable.

@drik98 drik98 force-pushed the feature/improve-backup-ux branch from e5615a4 to 4ee8a60 Compare February 20, 2026 14:31
@drik98
Copy link
Contributor Author

drik98 commented Feb 20, 2026

@schrieveslaach I think we will also have to remove the Logs link of the service of an back-upped app right? Currently we can still open the logs window but the API returns 400.

If you are so kind, pleas, go ahead. Sounds reasonable.

Changes were a bit more complicated then I imagined. I added some unit tests to bring some clarity about the logic when we actually render the expansion panel.

@drik98 drik98 force-pushed the feature/improve-backup-ux branch from 4ee8a60 to c333b02 Compare February 20, 2026 16:27
@drik98 drik98 changed the base branch from automatic-back-up to master February 20, 2026 16:28
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