-
Notifications
You must be signed in to change notification settings - Fork 527
Triaging process
This wiki outlines the steps to triage new and existing issues. This ensures that all issues are being effectively tracked and prioritized.
The Dev Workflow team lead (currently Ben) should do the following every week for any filed issues that aren’t assigned to a project:
- Check that the issue is written clearly enough. Request clarification if needed.
- Assign the issue to the appropriate task force, if applicable. Otherwise, assign it to the relevant team (CLAM, Dev Workflow) based on which type of user it affects.
- If you’re not sure how to categorise an issue, feel free to ping Sean (@seanlip) for help!
Team leads should do an audit every week to categorise new issues in their GitHub Projects board, following these steps:
-
Verify that the issue lies within the scope of the team (see definitions below).
- If not, assign the issue to the appropriate project/team.
-
Determine whether the issue is a bug or a enhancement
- If the issue is a feature request (enhancement), contact the relevant PM (product manager) for the team and ask them to take a look and decide whether it needs to be worked upon. (See the list of PMs below.)
- If the issue is a bug, add the Bug label.
-
(For simple UI issues) Make sure the bug is reproducible
- The issue should be consistently reproducible on either develop or the test/backup server.
- If that isn’t the case, ask the issue author for clarity.
- If the issue is not a simple UI bug that can be easily verified, the task of verifying the reproducibility lies on the developer working on the issue.
-
Once the Bug or Enhancement label has been added, the team lead should add the appropriate ‘Impact’ label (asking the PM or tech lead for help if they’re not sure), as well as the appropriate ‘Work’ label to the issue.
-
‘Impact’ defines how much the issue affects the functionality of the user-facing features. There are 3 associated labels: Impact: High, Impact: Medium and Impact: Low. For example, the issue is considered to have a high impact if it breaks a critical feature or blocks a release. General bugs (non-blocking) and improvements can be considered to be of medium impact while issues that occur in a few edge cases or affect a small number of users are issues with a low impact.
- Severity:
- Severe: Blocks or represents a primary user journey – e.g. end user playing a lesson, developer installing Oppia-Android.
- Moderate: blocks or represents a secondary user journey – e.g. end user changing a profile picture, developer easily seeing the output of frontend tests at the bottom of the CI log.
- Minor: Neither of the others, relates to “polish”.
- Severity:
-
‘Work’ defines how hard it would be to solve/resolve a particular issue. This is generally correlated with the amount of ambiguity in the task. We use the following rough system for assigning this label:
- Work: Low: Solution is clear and broken into good-first-issue-sized chunks.
- Work: Medium: Otherwise, if the means to find the solution is clear.
- Work: High: Everything else.
-
-
Once the appropriate ‘Impact’ and ‘Work Required’ labels has been added, determine its severity and assign the fitting priority label based on its impact (and possibly the work required label)
-
If the issue is a high-impact bug, put it in the high-priority bucket and prioritise it within the team.
-
For the priority bucket, team leads should include “important” issues – typically, a selection of high-impact enhancements and low-work medium-impact bugs – while keeping the size of that bucket about 2-3 times the projected stable number of team members.
Note This exercise might also help determine how large the team needs to be.
-
Other issues go in the ‘backlog’ bucket.
-
Any issue with an external dependency that the team cannot resolve by itself (example: missing mocks) goes in the ‘blocked’ bucket, and the reason should be specified in the ‘Issue specific status’ field (using free-text). Team leads should ensure an issue spends as little time here as possible.
-
-
Determine if the issue is potentially a good first issue
- If the issue seems like something a new contributor to Oppia-Android can take up, make sure to add enough information in the issue description for a new contributor to start work on that issue then ask Ben or Adhiambo to add the Good First Issue label. (Note: typically these are “Work: Low” issues.)
- For such good first issues, also assign purple “technology: Kotlin”, “technology: Bazel”, “technology: Xml”, etc. labels as appropriate. This helps new contributors identify issues that they would find approachable.
- If the issue seems like something a new contributor to Oppia-Android can take up, make sure to add enough information in the issue description for a new contributor to start work on that issue then ask Ben or Adhiambo to add the Good First Issue label. (Note: typically these are “Work: Low” issues.)
-
After triaging all issues, make sure the issues in the 'Priority' and 'High Priority' buckets are sorted based on severity/priority (high to low).
-
CLaM:
- Lead: Sean Lip (@seanlip)
-
Dev workflow:
- Lead: Ben Henning (@BenHenning)
Have an idea for how to improve the wiki? Please help make our documentation better by following our instructions for contributing to the wiki.
Core documentation
Developing Oppia
- Contributing to Oppia Android
- Bazel
- Key Workflows
- Testing
- Developing Skills
- Frequent Errors and Solutions
- RTL Guidelines
- Working on UI
- Writing Design Docs
Developer Reference
- Code style
- Background Processing
- Dark mode
- Buf Guide
- Firebase Console Guide
- Platform Parameters & Feature Flags
- Work Manager
- Dependency Injection with Dagger
- Revert & regression policy
- Upgrading target SDK version
- Spotlight Guide
- Triaging Process
- Bazel
- Internationalization
- Terminology in Oppia
- Past Events