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.

Sign up for GitHub

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

Jump to bottom

Add validation to group name. #66

Merged

kaishuu0123 merged 1 commit into kaishuu0123:master from yuuu:add_validation_to_group_name

Jan 14, 2020

Contributor

yuuu commented Jan 11, 2020

I found an issue where adding a group without a name made it impossible to open '/ application_settings/user_managements'.

So I am fixing the following 4 points.

Group name is required
Change the rendering destination at validation error from edit to index
Set @groups when creating
Change 'group' in index.html.erb to '@group'


          Add validation to group name.

4c609dc

kaishuu0123 merged commit c58cd29 into kaishuu0123:master

kaishuu0123 mentioned this pull request

Merged

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

Labels

None yet