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

Fix CongressMessage#common_fields #771

Open
esoterik opened this issue Oct 30, 2019 · 0 comments
Open

Fix CongressMessage#common_fields #771

esoterik opened this issue Oct 30, 2019 · 0 comments

Comments

@esoterik
Copy link
Member

Due to forcing some congress forms fields to be "always common" (e.g. showing up at the top of the form and being sent to the controller as common fields), fields with mismatching options_hash values (such as using state abbreviations instead of full state names in addresses) still get grouped together.

We should figure out a more consistent way of sharing the common fields between the controller and the congress message.

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

No branches or pull requests

1 participant