Add pending merge request badge count
Closes #6811
Feature - Added a bubble to the avatar and then a entry + bubble into the sub-navigation within the hamburger.
Technical
- The bubble is only visible when
admin/super-librarianshave atleast one pending merge requests.
Testing
https://gist.github.com/jimchamp/dd83ae3fe1ea86ae9621707fa96b4edf Execute these commands and you can see the bubble.
Screenshot
Avatar
Hamburger List
Stakeholders
Thanks for submitting this contribution!
@mekarpeles What does it mean by Needs: Submitter Input? Do I need to specify something? This is my first open source contribution so I don't know the process.
@mekarpeles What does it mean by
Needs: Submitter Input? Do I need to specify something? This is my first open source contribution so I don't know the process.
Staff members typically add the Needs: Submitter Input label to PRs that have been reviewed, but need some changes from the contributor. I think that we didn't notice that you pushed the requested changes, and added the label in error. Apologies!
Noticed a few small improvements that could be made since I last reviewed this. Those changes have been committed. Thanks for your contribution, @SyedMa3!
@jimchamp Thanks for patiently helping and suggesting improvements.