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 #1037 - Removes dead code and method. #1555

Merged
merged 1 commit into from
May 11, 2017

Conversation

karlcow
Copy link
Member

@karlcow karlcow commented May 11, 2017

This removes dead code on the home page. Basically we never use this code anymore since we completely transitioned to /issues/new for creating a new issue.

To check if add-on is hitting "/" instead of "/issues/new" @miketaylr will know

r? @miketaylr

@karlcow karlcow requested a review from miketaylr May 11, 2017 01:11
@karlcow karlcow changed the title Issue #1037 - Removes dead code and method. Fixes #1037 - Removes dead code and method. May 11, 2017
@miketaylr
Copy link
Member

miketaylr commented May 11, 2017

To check if add-on is hitting "/" instead of "/issues/new" @miketaylr will know

If people have older versions installed they will land at /?open=1&url=blah... but the form POST action will always be /issues/new, so it's a non-issue IMO.

@miketaylr
Copy link
Member

r+ thanks @karlcow!

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