Skip to content

Conversation

@rfontanarosa
Copy link
Collaborator

@rfontanarosa rfontanarosa commented Oct 1, 2025

closes #2122

Screenshot 2025-11-12 alle 12 02 57 Screenshot 2025-11-12 alle 12 03 05

@rfontanarosa rfontanarosa self-assigned this Oct 1, 2025
@rfontanarosa rfontanarosa changed the title Allow Survey Organizers to clone surveys Allow to clone surveys Oct 1, 2025
@rfontanarosa rfontanarosa changed the title Allow to clone surveys Allow to copy surveys Oct 3, 2025
@rfontanarosa
Copy link
Collaborator Author

@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

Screenshot 2025-10-10 alle 13 18 40

@gino-m
Copy link
Collaborator

gino-m commented Oct 10, 2025

@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.

What happens if you create the survey first in the transaction?

I'm not sure about the text and if name this functionality Clone survey or Copy survey

"Copy" is much more common and comprehensible.

@rfontanarosa rfontanarosa marked this pull request as ready for review October 22, 2025 10:29
@rfontanarosa rfontanarosa requested a review from gino-m October 22, 2025 10:32
@rfontanarosa rfontanarosa requested a review from gino-m November 12, 2025 11:00
@gino-m gino-m changed the title Allow to copy surveys Add "Copy survey" feature Nov 14, 2025
} as Survey);
}

async copySurvey(surveyId: string): Promise<string> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Tests please.

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.

Allow users to copy existing surveys

3 participants