flagr
flagr copied to clipboard
Display the key for each flag
Description
Addresses #424. Displays key in both the flag list and the deleted flag list. This makes it easier/more convenient to use the key as the name of the flag.
Motivation and Context
See above.
How Has This Been Tested?
I don't see tests for the frontend, but it did pass npm run lint
. I tested it simply by running the app locally. See the new column:
Types of changes
- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
- [x] My code follows the code style of this project.
- [ ] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
- [ ] I have added tests to cover my changes.
- [ ] All new and existing tests passed.
Codecov Report
Merging #425 (badd4c5) into master (c3dc3c0) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #425 +/- ##
=======================================
Coverage 81.75% 81.75%
=======================================
Files 27 27
Lines 1611 1611
=======================================
Hits 1317 1317
Misses 214 214
Partials 80 80
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update c3dc3c0...badd4c5. Read the comment docs.
Stale pull request message