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

Remove question about learner count from WRF #2523

Conversation

elichad
Copy link
Contributor

@elichad elichad commented Aug 21, 2023

Fixes #2308 by completely removing this field from the DB (issue comments say that this data is not used)

@elichad elichad marked this pull request as draft August 21, 2023 13:29
@elichad elichad removed the request for review from pbanaszkiewicz August 21, 2023 13:29
@elichad
Copy link
Contributor Author

elichad commented Aug 21, 2023

Converting to draft while I figure out how to make the migration reversible

@elichad elichad marked this pull request as ready for review August 21, 2023 16:14
Copy link
Contributor

@pbanaszkiewicz pbanaszkiewicz left a comment

Choose a reason for hiding this comment

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

LGTM. Interesting case with the alter field before it is removed through migration.

@elichad elichad merged commit 74708de into develop Aug 23, 2023
8 checks passed
@danielle06
Copy link

@elichad , saw the original pr was closed. Wasn't sure if that meant that the count field was able to be removed - double checking because it still shows on the form.

@elichad
Copy link
Contributor Author

elichad commented Sep 12, 2023

@danielle06 Sorry for the confusion! The PR is closed as the development is complete, but this change hasn't been released yet. The question should disappear from the form after the next release (v4.3, planned for late October). Is that okay?

@danielle06
Copy link

@elichad , yes! No worries and thank you for the explanation. I wanted to make sure I was understanding correctly :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove question about learner count from workshop request form
3 participants