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

Pascal/no edit on published version #55

Merged
merged 4 commits into from
Jul 13, 2023

Conversation

Pascal-Delange
Copy link
Contributor

@Pascal-Delange Pascal-Delange commented Jul 12, 2023

Don't allow editing a scenario iteration's thresholds if the version is published

Capture d’écran 2023-07-12 à 14 27 28

@balzdur balzdur changed the base branch from main to builder July 12, 2023 12:03
@balzdur balzdur changed the base branch from builder to composable-combobox July 12, 2023 12:04
@Pascal-Delange Pascal-Delange force-pushed the pascal/no-edit-on-published-version branch from 42e42be to de08cdb Compare July 12, 2023 13:33
@balzdur balzdur force-pushed the pascal/no-edit-on-published-version branch from de08cdb to d262b43 Compare July 12, 2023 16:55
@balzdur balzdur force-pushed the pascal/no-edit-on-published-version branch from d262b43 to 37a2ecc Compare July 13, 2023 14:21
@balzdur balzdur changed the base branch from composable-combobox to main July 13, 2023 14:22
Copy link
Collaborator

@balzdur balzdur left a comment

Choose a reason for hiding this comment

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

Rebase + move "edit only draft" rule to the edit parent <Outlet />

@balzdur balzdur enabled auto-merge July 13, 2023 14:22
@balzdur balzdur force-pushed the pascal/no-edit-on-published-version branch from 37a2ecc to 9da6f1d Compare July 13, 2023 16:08
@balzdur balzdur merged commit 1e0a6f3 into main Jul 13, 2023
1 check passed
@balzdur balzdur deleted the pascal/no-edit-on-published-version branch July 13, 2023 16:11
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