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

Fixes #6538 - User group vs usergroup name consistency #1710

Closed
wants to merge 1 commit into from
Closed

Fixes #6538 - User group vs usergroup name consistency #1710

wants to merge 1 commit into from

Conversation

boaz0
Copy link
Member

@boaz0 boaz0 commented Aug 24, 2014

I haven't seen any inconsistency in the app/view/*. However, I have changed to user group in app/controllers/api/v1/usergroups_controller.rb and app/models/auth_sources/auth_source_ldap.rb.

@orrabin
Copy link
Member

orrabin commented Aug 24, 2014

[test]

1 similar comment
@orrabin
Copy link
Member

orrabin commented Aug 24, 2014

[test]

@boaz0
Copy link
Member Author

boaz0 commented Aug 25, 2014

What the devil?

Resolving dependencies...
Could not find gem 'rails (= 3.2.18) ruby' in the gems available on this
machine.
Build step 'Execute shell' marked build as failure

Is this my fault?

@orrabin
Copy link
Member

orrabin commented Aug 25, 2014

@ripcurld00d it's not your'e fault all of the tests failed.
[test] again

@boaz0
Copy link
Member Author

boaz0 commented Aug 25, 2014

@orrabin i see :) thanks.

@domcleal
Copy link
Contributor

There's a "Usergroup" on the first form tab, stored in app/views/usergroups/_form.html.erb.

@boaz0
Copy link
Member Author

boaz0 commented Aug 26, 2014

Yes, you're right. I've also found some in views/usergroups/index.html.erb.
I used a wrong regex when running grep.
Anyhow, I will fix this and squash.

@boaz0
Copy link
Member Author

boaz0 commented Aug 26, 2014

I am not sure what s_ does. Is it okay to replace s_("Usergroup|Name") with s_("User group|Name")?

@orrabin
Copy link
Member

orrabin commented Aug 27, 2014

[test]

@domcleal
Copy link
Contributor

Don't change the s_() calls, what happens in these is that s_() returns the section after the | but translated - users will never see the first part. The "Usergroup" in those refers to the table name, which isn't changing, so just keep them as-is.

@boaz0
Copy link
Member Author

boaz0 commented Aug 27, 2014

Uh huh. OK.

@orrabin
Copy link
Member

orrabin commented Aug 28, 2014

[test]

@isratrade
Copy link
Member

merged as f45717f. Thanks @ripcurld00d !

@isratrade isratrade closed this Sep 2, 2014
@boaz0 boaz0 deleted the fix6538_patch branch September 23, 2014 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants