Skip to content

Fixes #975 - Replace new with needstriage#1114

Merged
miketaylr merged 2 commits intowebcompat:masterfrom
deepthivenkat:issues/975/1
Jul 6, 2016
Merged

Fixes #975 - Replace new with needstriage#1114
miketaylr merged 2 commits intowebcompat:masterfrom
deepthivenkat:issues/975/1

Conversation

@deepthivenkat
Copy link
Copy Markdown
Member

Created a new pull request for all the previous commits in #975 from my local fork.

The functional test "Issue-list - Loading URL with stage param loads issues" passed in my local machine.
Raised a separate issue because this fails in Travis.

All other functional test cases passed successfully.

@deepthivenkat
Copy link
Copy Markdown
Member Author

Opened Issue #1115

# For paginated results on the /issues page, see /issues/search/new.
# Note that 'needstriage' here is primarily used on the homepage.
# For paginated results on the /issues page,
# see /issues/search/needstriage.

This comment was marked as abuse.

@deepthivenkat
Copy link
Copy Markdown
Member Author

All the tests passed in my local machine. Whereas most tests failed in Travis.

photo593046103980615620

@miketaylr
Copy link
Copy Markdown
Member

Whereas most tests failed in Travis.

Yeah, that's because your PR is from a fork which means Travis doesn't give it access to secret env variables. Thanks for showing that tests pass for you locally!

@miketaylr
Copy link
Copy Markdown
Member

r+, looks good!

@miketaylr miketaylr merged commit 99efc78 into webcompat:master Jul 6, 2016
@miketaylr
Copy link
Copy Markdown
Member

This is kind of weird, Travis is failing tons of tests -- but locally it's OK for me:

https://travis-ci.org/webcompat/webcompat.com/builds/142773957

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