Skip to content
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

Migration: Add foorm_misc_survey table #35439

Merged
merged 4 commits into from Jun 24, 2020

Conversation

molly-moen
Copy link
Contributor

@molly-moen molly-moen commented Jun 23, 2020

We are adding a lightweight foorm survey handler for one-off surveys. We need a new metadata table to store user information for these surveys. The new table, foorm_misc_survey will contain:

foorm_submission_id
user_id
misc_form_path
timestamps

Where misc_form_path represents the path to the survey, which may be different from the foorm name. This is because misc_surveys may include built-in default parameters, so multiple paths may point to the same form in the backend.

Links

Testing story

Validated new table works as expected.

Reviewer Checklist:

  • Tests provide adequate coverage
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@molly-moen molly-moen requested review from breville and a team June 23, 2020 15:51
@molly-moen molly-moen merged commit e603225 into staging Jun 24, 2020
@molly-moen molly-moen deleted the molly/misc-survey-foorm-migration branch June 24, 2020 18:54
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.

None yet

2 participants