Skip to content

[TAN-8062] Allow configuring multiple submissions per user for native survey phases#14169

Open
amanda-anderson wants to merge 9 commits into
masterfrom
TAN-8062-multiple-survey-submissions
Open

[TAN-8062] Allow configuring multiple submissions per user for native survey phases#14169
amanda-anderson wants to merge 9 commits into
masterfrom
TAN-8062-multiple-survey-submissions

Conversation

@amanda-anderson

@amanda-anderson amanda-anderson commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Note

Rob may have some small suggestions for UX improvements, but I think this can probably be code reviewed now in any case 🤔

Changelog

Added

  • [TAN-8062] Allow configuring multiple submissions per user for native survey phases

@notion-workspace

Copy link
Copy Markdown

@cl-dev-bot

cl-dev-bot commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator
Messages
📖 Changelog provided 🎉
📖 Notion issue: TAN-8062
📖

Run the e2e tests

📖 Check translation progress

Generated by 🚫 dangerJS against b57c863

@amanda-anderson amanda-anderson requested a review from adessy June 24, 2026 14:09

@adessy adessy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me! A couple of thoughts:

  • Could it make sense to expose this setting in other participation methods too(say, for restricting posting in ideation for instance)?
  • The spacing and styling of the toggles feel a bit inconsistent across the phase config pages. Might be worth a small cleanup pass at some point to align them.

Comment thread back/lib/participation_method/native_survey.rb Outdated
amanda-anderson and others added 5 commits July 1, 2026 09:17
…nts/PhaseParticipationConfig/components/inputs/NativeSurveyInputs.tsx

Co-authored-by: adessy <209268+adessy@users.noreply.github.com>
Co-authored-by: adessy <209268+adessy@users.noreply.github.com>
@amanda-anderson amanda-anderson requested a review from adessy July 6, 2026 06:31
@amanda-anderson

Copy link
Copy Markdown
Contributor Author

@adessy I've extended this as suggested to the other phase types :) There were quite a few code changes though - so I think another code review might be smart? 🙏

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.

3 participants