Add link to merge queue for librarians (like we have for super librarians)

[RayBB]

Proposal

Right now super librarians can see a button for merge queue in the menu. It's quite handy.
I propose we make the button visible for librarians too. Otherwise it's not easy to get to the merge queue to see how your merge requests are progressing.

What super librarians see:

What regular librarians see:

Justification

Breakdown

Requirements Checklist

• [ ]

Related files

Stakeholders

• @seabelis

---

Instructions for Contributors

Please run these commands to ensure your repository is up to date before creating a new branch to work on this issue and each time after pushing code to Github, because the pre-commit bot may add commits to your PRs upstream.

[AST0008]

@RayBB hey, can i work on this?

[RayBB]

@AST0008 yes. Please be sure to read the contributing guide and add screenshots to your PR.

[PredictiveManish]

If I worked on this can I open a PR?

[RayBB]

@AST0008 are you still working on this?
If we don't hear back from AST0008 by Monday then you can open your PR @PredictiveManish

[PredictiveManish]

@AST0008 are you still working on this? If we don't hear back from AST0008 by Monday then you can open your PR @PredictiveManish

Sure.

[ananyakaligal]

@AST0008 are you still working on this?

[AST0008]

@ananyakaligal you can work on this, I'm currently busy with something else

[RayBB]

@PredictiveManish I assigned you since you asked first. Then @ananyakaligal is next in line :)
Thanks for keeping us updated @AST0008

[mekarpeles]

@PredictiveManish

Before you go too deep into trying to solve the issue, can you please first help us with breakdown by:

1. Clarifying what you believe the task to be
2. Identifying which files you believe may be related from the project

• as a freebie, I did a search for "Pending Merge Requests" -- the item we want to show up for more patrons -- and found this match:

  openlibrary/openlibrary/templates/lib/nav_head.html

  Line 15 in 244b063

  $ loginLinks.insert(2, { "href": homepath() + "/merges", "text": _("Pending Merge Requests"),"track": "MyRequests", "badge": True })

3. Asking any question(s) you may have about the goal or requirements of this issue
4. Proposing a solution or approach

Thank you for your help!

[andycandy]

I'd love to work on this issue if its okay.

[PredictiveManish]

@PredictiveManish I assigned you since you asked first. Then @ananyakaligal is next in line :) Thanks for keeping us updated @AST0008

Thanks for giving me opportunity to work.

[PredictiveManish]

Here the librarians can also be allowed and a simple UI Change might be solving this issue completely.
For solving this issue:
This two files can be changed a little bit to allow librarians to see the pending merge requests option and also they can be able to merge from their side.
First here the librarian can be added to showing pending merge requests button:

openlibrary/openlibrary/templates/lib/nav_head.html

Line 2 in 244b063

$ is_privileged_user = ctx.user and (ctx.user.is_admin() or ctx.user.is_super_librarian())

Then the logic of allowing library will help to overcome the issue with using this file.
Also,

openlibrary/openlibrary/components/MergeUI.vue

Line 78 in 244b063

isSuperLibrarian() {

:

like here it is allowed for superlibrarians similar can be done for librarians too.

[RayBB]

@PredictiveManish Only the first change is needed. Add is_librarian and that should be good to go!

The second one is for the actual merges instead of seeing the merge queue.

Please open a PR for this.