Skip to content

Changed module name#9

Open
davbeck wants to merge 5 commits intothecity:masterfrom
davbeck:master
Open

Changed module name#9
davbeck wants to merge 5 commits intothecity:masterfrom
davbeck:master

Conversation

@davbeck
Copy link

@davbeck davbeck commented Oct 29, 2013

The module name was conflicting with "thecity" user api gem.

Fixes Issue #7

In order to not conflict the the user api gem, which also uses "TheCity", we use a different module name to make the classes unique.
Conflicts:
	spec/api/group_address_list_spec.rb
	spec/api/group_checkin_list_spec.rb
	spec/api/group_event_attendance_list_spec.rb
	spec/api/role_list_spec.rb
	spec/api/skill_list_spec.rb
	spec/api/skilled_user_id_list_spec.rb
	spec/api/skilled_user_list_spec.rb
	spec/api/tag_group_list_spec.rb
	spec/api/tag_list_spec.rb
	spec/api/terminology_list_spec.rb
	spec/api/user_address_list_spec.rb
	spec/api/user_admin_privilege_list_spec.rb
	spec/api/user_family_list_spec.rb
	spec/api/user_invitation_list_spec.rb
	spec/api/user_list_spec.rb
	spec/api/user_note_list_spec.rb
	spec/api/user_process_answer_list_spec.rb
	spec/api/user_process_list_spec.rb
	spec/api/user_role_list_spec.rb
	spec/api/user_skill_list_spec.rb
	spec/api/user_spec.rb
	spec/api/web_hook_list_spec.rb
	spec/readers/api_reader_spec.rb
@TheMetalCode
Copy link
Contributor

Hey David, can you go ahead and merge the latest origin/master into your branch and push again? Reason being, tests and TravisCI actually work now, so we'd know whether or not the module name change is safe. I'm sure it is, of course, but to be sure. ;)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants