Skip to content

fix: allow username to be omitted - #207

Open
mesemus wants to merge 1 commit into
inveniosoftware:masterfrom
oarepo:fix-obligatory-username
Open

fix: allow username to be omitted#207
mesemus wants to merge 1 commit into
inveniosoftware:masterfrom
oarepo:fix-obligatory-username

Conversation

@mesemus

@mesemus mesemus commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Description

  • Commit cff2f68 added username validation, effectively making username required (but None could be passed)
  • In other parts of Invenio, username is treated as optional.
  • This commit restores that behavior by making username optional in this validation check.

Checklist

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

Frontend

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.

* Commit cff2f68 added username validation,
  effectively making username required
  (but None could be passed)
* In other parts of Invenio, username is
  treated as optional.
* This commit restores that behavior by
  making username optional in this
  validation check.
@mesemus mesemus added this to v14 Jun 17, 2026
@mesemus mesemus moved this to Triage in v14 Jun 17, 2026

@tmorrell tmorrell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a good bug fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

3 participants