-
Notifications
You must be signed in to change notification settings - Fork 7
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
Allow user to create, iterate and publish/unpublish a scenario #73
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks about right to me so far ! 👍
On thing we should do: you should borrow from @zoeKD 's PR to display the relevant buttons only when they should be available. (well, you have to go through a rebase of master anyway because Zoe's changes created quite a few conflicts in your PR)
i18n: ['scenarios', 'navigation', 'common'] satisfies Namespace, | ||
}; | ||
|
||
const createListFormSchema = z.object({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
createScenarioFormSchema?
Create button to delete rule Make publish work Edit rule information
Allow moving around different iteration Clean some code run prettier
…ditor If draft exist change modal to give choice to switch to it or override it for creating draft Rename Score to Score modifier in rule Add min max value from -1000 to 1000 to score modifier in rule and display error for it
136e77f
to
b683f3f
Compare
Co-authored-by: Pascal <128643171+Pascal-Delange@users.noreply.github.com>
change translation for when there is no rule
Functionality added:
Miscellaneous: