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

Not clear which fields are required #1089

Merged
merged 2 commits into from Jul 23, 2013
Merged

Not clear which fields are required #1089

merged 2 commits into from Jul 23, 2013

Conversation

johnmartin
Copy link
Contributor

When creating a dataset, required fields should be clearly marked, e.g. with a red *. Also when creating a new user ID.

@johnmartin
Copy link
Contributor

This should be site wide as well.

@ghost ghost assigned johnmartin Jul 16, 2013
* Adds extra params to form macros called `is_required`
* Also adds extra macro for outputting the '* is required field' on the bottom
of forms
* Also adds said params to needed forms
@ghost ghost assigned tobes Jul 23, 2013
@tobes
Copy link
Contributor

tobes commented Jul 23, 2013

@johnmartin can you add the is_required param to the documentation in macros/form.html else looks good

@ghost ghost assigned johnmartin Jul 23, 2013
@johnmartin
Copy link
Contributor

@tobes There you go.

@ghost ghost assigned tobes Jul 23, 2013
@tobes tobes merged commit 1c3604d into master Jul 23, 2013
@smotornyuk smotornyuk deleted the 1089-required-fields branch December 19, 2018 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants