Skip to content
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.

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

Make system-generated Plan Snapshots provide Name and Description #1592

Open
Mythicaeda opened this issue Nov 5, 2024 · 0 comments
Open
Labels
database Anything related to the database

Comments

@Mythicaeda
Copy link
Contributor

Mythicaeda commented Nov 5, 2024

Currently, system-generated snapshots made during the plan branch-merge workflow are unnamed. Since the UI is choosing to display these snapshots, we ought to change this behavior.

Between the names and description, we should provide the source plan's name and the action that triggered the snapshot. The generated name requires some type of UUID to avoid raising issues with the constraint that prevents duplicate snapshot names for a plan (potentially the timestamp the snapshot was taken at). The name generation needs a failsafe -- like "[plan name] at [timestamp] (2)".

We may want to discuss declaring a system user to go in the taken_by field, otherwise NULL is still a good way to identify snapshots not taken by an actual user.

@Mythicaeda Mythicaeda added the database Anything related to the database label Nov 5, 2024
@Mythicaeda Mythicaeda added this to Aerie Nov 5, 2024
@github-project-automation github-project-automation bot moved this to Todo in Aerie Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
database Anything related to the database
Projects
Status: Todo
Development

No branches or pull requests

1 participant