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

Delete 'Others' section from hits #101

Merged
merged 1 commit into from Dec 5, 2013
Merged

Delete 'Others' section from hits #101

merged 1 commit into from Dec 5, 2013

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Dec 3, 2013

  • Others confused all users in research
  • The list cannot be acted on or resolved from within transition alone
  • Attempts to create mappings for these statuses will put the system in
    a confusing state (eg. for robots.txt)
  • Remove references to Other from all tests, models, controllers and
    views
  • All hit information remains available under 'All hits'
* Others confused all users in research
* The list cannot be acted on or resolved from within transition alone
* Attempts to create mappings for these statuses will put the system in
a confusing state (eg. for robots.txt)
* Remove references to Other from all tests, models, controllers and
views
@jamiecobbett
Copy link
Contributor

@jamiecobbett jamiecobbett commented Dec 3, 2013

The example of robots.txt is something that should be solved soon: https://www.pivotaltracker.com/story/show/58806284 does that affect this PR?

@fofr
Copy link
Contributor Author

@fofr fofr commented Dec 3, 2013

I don't think it affects it, it would lessen some of the confusion and risk around the section, but it'd still be misinterpreted and remain of limited value.

@rgarner
Copy link
Contributor

@rgarner rgarner commented Dec 3, 2013

This PR is full of 🔴

I like 🔴

jamiecobbett added a commit that referenced this pull request Dec 5, 2013
Delete 'Others' section from hits
@jamiecobbett jamiecobbett merged commit 9547478 into master Dec 5, 2013
@jamiecobbett jamiecobbett deleted the others-begone branch Dec 5, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.