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

#24 Enhancements vs bugs #25

Merged
merged 3 commits into from
Jan 17, 2018
Merged

#24 Enhancements vs bugs #25

merged 3 commits into from
Jan 17, 2018

Conversation

Cleop
Copy link
Member

@Cleop Cleop commented Jan 16, 2018

#24 Write up about when to create a new issue for an enhancement and when to keep comments in the same issue for a bug

@Cleop Cleop assigned Cleop and iteles and unassigned Cleop Jan 16, 2018
@Cleop Cleop requested a review from iteles January 16, 2018 12:10
Copy link
Member

@iteles iteles left a comment

Choose a reason for hiding this comment

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

Added some comments @Cleop, happy to merge this in and have you deal with them separately if it's less of a hinderance to the flow?

An existing issue is put into `please-test` as the team have finished working on
it and all of the acceptance criteria have been fulfilled. You, the product
owner test the issue and on reflection you decide that you think the designs
would look better with a couple of tweaks - maybe removing the bold on the
Copy link
Member

Choose a reason for hiding this comment

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

This makes me think that we could probably do with a flow diagram of what an issue goes through and the various actors.

fundamentally not what you asked for in the first place or not what was shown on
the mock-up then they belong in a new issue **NOT** as a 'bug' or as a request
at the bottom of the existing issue. Small as they seem, these issues are
enhancements and new scope and they should be dealt with separately.
Copy link
Member

Choose a reason for hiding this comment

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

We should look to add an example here of what's an enhancement

readme.md Outdated
of times or if there are a few across multiple issues. Not only do the github
issues become muddied as their title may no longer reflect the content inside
but also it becomes more complicated for people to follow the chains of
conversations that have evolved. Also, the extra changes will impact the
Copy link
Member

Choose a reason for hiding this comment

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

Got a little cross-eyed reading this big block of text - it could do with some additional highlighting with some boldening of important words and phrases . We know POs are unlikely to read everything here!
dwyl/start-here#118 (comment)

readme.md Outdated
### So what changes count as bugs :bug: then?
Bugs are places where the acceptance criteria has not been met :negative_squared_cross_mark: :

ie. you were meant to change the font size across all titles on this page but
Copy link
Member

Choose a reason for hiding this comment

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

This comment is a little pedantic, so feel free to ignore it, but strikes me as more of an e.g. than an id est 😊

readme.md Outdated

**OR**

Places where the changes made in an issue have broken something else that
Copy link
Member

Choose a reason for hiding this comment

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

Happy to have this in here but it could get messy - if the cancel button colour is changed on another page when you're only supposed to change the cancel button colour on this page, do you open a new bug issue for the other page or do you add it to the current issue which is not relevant to the other page?

@Cleop Cleop assigned iteles and unassigned Cleop Jan 16, 2018
readme.md Outdated
this example there are 3 actors, the Product Owner (PO), the Scrum Master
(SM) and the developer (Cleop).

<img src="https://user-images.githubusercontent.com/16775804/35003068-8d01f586-fae3-11e7-9e35-784092b42d62.png" width=350px />
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for adding @Cleop!
A couple of notes on this:

  • It should be the development team that adds the time estimate (sometimes the SM but best practice is really the dev team)
  • For step 8, I would suggest that you create a fork in the flow - either closing the issue if satisfied (most likely scenario) or if necessary reporting a bug or opening a new issue

Copy link
Member

Choose a reason for hiding this comment

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

Worth also adding a link to our contributing process here for anyone who's curious?

@Cleop
Copy link
Member Author

Cleop commented Jan 17, 2018

@iteles - I've added the contributing link, perhaps you can take a look at my redraft of the issue flow?

Copy link
Member

@iteles iteles left a comment

Choose a reason for hiding this comment

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

Might be worth taking some of the 'noise' out of that issue screenshot we have in here for an example of when to open a new issue (could be done pretty simply in Preview even), but not a blocker. Thanks @Cleop!

@iteles iteles merged commit 8d960a1 into master Jan 17, 2018
@iteles iteles deleted the bugs-enhancements branch January 17, 2018 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants