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

Update urls.py #98

Merged
merged 1 commit into from
May 13, 2015
Merged

Update urls.py #98

merged 1 commit into from
May 13, 2015

Conversation

jimlyndon
Copy link
Contributor

Removed trailing slash to resolve tastypie endpoints correctly (not relying on append slash/middleware).
See issue #89

removed trailing slash to resolve tastypie endpoints correctly (not relying on append slash).
See issue concentricsky#89
@yusun0715
Copy link

Hey Jim, I just found out today that the actual schema url doesn't match with the its definition (one more slash there). I am so happy to see it resolved. Thanks for doing that.

I may be curious about when you guys will merge this PR and do another deployment so that we can have our tastypie_swagger up-to-date.

Best regards,

johnraz added a commit that referenced this pull request May 13, 2015
@johnraz johnraz merged commit c7f023a into concentricsky:master May 13, 2015
@johnraz
Copy link
Contributor

johnraz commented May 13, 2015

I can merge but not deploy ;-)

@yusun0715
Copy link

It is alright. Thanks for doing that!
I will keep my four eyes on its future deployment. :)

Yu Sun
Back-end Developer
100 Franklin Street, Suite 403, Boston, MA 02110, United Sates of America
Direct: +1 508-414-5740 | Office: +1 857 214 7340 | Skype:
yu.sun.polestar
[email protected] [email protected] |
www.polestarglobal.com

The Pole Star group of companies includes Pole Star Space Applications
Ltd., Pole Star Data Centre Services Ltd.,
Absolute Software Inc., Absolute Maritime Tracking Services S.A.
and Absolute Software (Australia) Pty. Ltd.



This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error and are not the named addressee
please notify the sender and delete this email from your system. If you are
not the intended recipient you are notified that disclosing, copying,
distributing or taking any action in reliance on the contents of this
information is strictly prohibited.

On 13 May 2015 at 15:40, Jonathan Liuti [email protected] wrote:

I can merge but not deploy ;-)


Reply to this email directly or view it on GitHub
#98 (comment)
.

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.

3 participants