Skip to content

Conversation

@mikeharv
Copy link
Contributor

@mikeharv mikeharv commented Jan 7, 2026

This adds a check that the translation for a project title exists before adding the current project type to the create header menu ("New Project +" menu).

@dju90 reported an issue where a broken menu button was adding when viewing a Mix&Move project outside of the curriculum:
image (372)

@molly-moen shared helpful context that "if a project type isn't in the list but you are on the standalone project, we try to put it in the menu."

A potential easy follow-up here is also adding a block or allow list which would effectively hide this particular project type if desired, even if we did have a translation string and thumbnail available. For now this just aims to fix the rendering issue by hiding the button.

Note that with this change other labs types can still be inserted to the top of the menu if we have a string:
image (373)

Links

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Creation Checklist:

  • Tests provide adequate coverage
  • Privacy impacts have been documented
  • Security impacts have been documented
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Follow-up work items (including potential tech debt) are tracked and linked

@mikeharv mikeharv changed the title Hide broken Mix&Move project button from header menu Jan 7, 2026
@mikeharv mikeharv requested review from a team and dju90 January 7, 2026 22:04
Copy link
Contributor

@hannahbergam hannahbergam left a comment

Choose a reason for hiding this comment

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

Makes sense to me!

@mikeharv mikeharv merged commit 6033646 into staging Jan 7, 2026
6 checks passed
@mikeharv mikeharv deleted the mike/hide-broken-project-buttons branch January 7, 2026 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants