-
Notifications
You must be signed in to change notification settings - Fork 98
Add "Copy survey" feature #2264
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
base: master
Are you sure you want to change the base?
Add "Copy survey" feature #2264
Conversation
|
@gino-m I wanted to move the logic within a transaction, but it returns a rules error because it attempts to create a job inside a survey that hasn't been saved yet. Therefore, I need to find a different approach or leave things as they are. I'm not sure about the text and if name this functionality Clone survey or Copy survey
|
What happens if you create the survey first in the transaction?
"Copy" is much more common and comprehensible. |
web/src/app/pages/edit-survey/edit-details/edit-details.component.html
Outdated
Show resolved
Hide resolved
| } as Survey); | ||
| } | ||
|
|
||
| async copySurvey(surveyId: string): Promise<string> { |
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.
Tests please.

closes #2122