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

Link to unit and section. #5103

Merged
merged 1 commit into from
Sep 8, 2014
Merged

Link to unit and section. #5103

merged 1 commit into from
Sep 8, 2014

Conversation

davestgermain
Copy link
Contributor

@ormsbee
Copy link
Contributor

ormsbee commented Sep 6, 2014

BTW, do you know what's up with the test failure?

@ormsbee
Copy link
Contributor

ormsbee commented Sep 8, 2014

Sorry I was unclear, can add the unit URL in addition to the existing section_url, instead of replacing it?

@davestgermain davestgermain changed the title Link to unit rather than section. Link to unit and section. Sep 8, 2014
@davestgermain
Copy link
Contributor Author

@ormsbee done. Tests are still failing, unfortunately.

ormsbee pushed a commit that referenced this pull request Sep 8, 2014
Link to unit and section.
@ormsbee ormsbee merged commit 9d92019 into ormsbee/mobile_api2 Sep 8, 2014
@davestgermain davestgermain deleted the dcs/MA-33 branch September 23, 2014 15:12
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