-
Notifications
You must be signed in to change notification settings - Fork 6
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
Feature/create new rs versions #386
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.
Seems to work well. We could maybe add a pill to each version in the dropdown to indicate if they have been published or not.
@@ -12,6 +14,33 @@ export default class RegulatoryAttachmentsShowController extends Controller { | |||
generateExportFromEditorDocument(this.model.editorDocument); | |||
} | |||
|
|||
@task | |||
*createNewVersion() { |
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.
this seems to duplicate quite a bit from https://github.com/lblod/frontend-gelinkt-notuleren/pull/385/files#diff-43a8e9478c35d069347a4336692d075b1fd351b364bb6cb2a579e13be02ba183R36 I'd move the logic to the document service and use that in both
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.
works well
No description provided.