Fix ESLint problems

[ptamarit]

❤️ Thank you for your contribution!

Description

Every pull request in this repository shows inline ESLint annotations for these 4 problems:

• 'SomeComponent' is defined but never used
• 'someVariable' is assigned a value but never used
• Do not use Array index in keys
• Fragments should contain more than one child - otherwise, there’s no need for a Fragment at all

This pull requests solves the 3 4 first issues.
The 4th issue will be fixed by inveniosoftware/eslint-config-invenio#16.

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

• I'm aware of the code of conduct.
• I've created logical separate commits and followed the commit message format.
• I've added relevant test cases.
• I've added relevant documentation.
• I've marked translation strings.
• I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
• I've NOT included third-party code (copy/pasted source code or new dependencies).
  • If you have added third-party code (copy/pasted or new dependencies), please reach out to an architect.

Frontend

• I've followed the CSS/JS and React guidelines.
• I've followed the web accessibility guidelines.
• I've followed the user interface guidelines.

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

1. You license your contribution under the same terms as the current repository’s license.
2. You agree that you have the right to license your contribution under the current repository’s license.

[ptamarit]

Just rebased to fix a conflict on ShareModal.js, where the unused import has been removed in the meantime.