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

Clarified instructions for opening issues on ancillary repos #43

Closed

Conversation

adamzap
Copy link
Contributor

@adamzap adamzap commented Apr 26, 2015

This might help to close #32.

I researched overriding Trac's error template to change the "Create" button's href, and it seems non-trivial. So it seems like two options available are:

  1. Use jQuery to rewrite the button's href on page load
  2. Clarify the instructions

This patch implements option 2.

Regardless of it's relation to #32, I think this patch is an improvement since it links to both issue trackers.

Before

screen shot 2015-04-25 at 11 47 14 pm

After

screen shot 2015-04-25 at 11 48 49 pm

@@ -28,8 +28,10 @@
<a href="https://docs.djangoproject.com/en/dev/internals/contributing/localizing/#translations">bug reports on translations</a>.
</li>
<li>
Please <a href="https://github.com/django/djangoproject.com/issues">create an issue on GitHub</a>
for issues with our web site (www.djangoproject.com) or with Trac (code.djangoproject.com).
Report issues with our web site (www.djangoproject.com) in <a href="https://github.com/django/djangoproject.com/issues">it's GitHub issue tracker</a>.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe it should be "its", not "it's" here.

@bmispelon
Copy link
Member

I need to check with the team if this is the actual policy we have (I remember it differently but it's been a while) but other than that, the PR looks good (well except for the two minor issues I pointed out).

Thanks!

@adamzap adamzap force-pushed the clarify-ancillary-repo-instructions branch from dbbfd8b to 071979f Compare April 26, 2015 14:13
@adamzap
Copy link
Contributor Author

adamzap commented Apr 26, 2015

Ok, sounds good!

I fixed the issues you mentioned. Thanks for the review!

@bmispelon
Copy link
Member

Merged in e0436e0.

Thanks! 🚀

@bmispelon bmispelon closed this May 1, 2015
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.

Trac recommends creating tickets to incorrect ticket tracker
2 participants