Skip to content

feat: add Github URL to navbar single type - #98

Merged
matej-marcisovsky merged 1 commit into
mainfrom
75-change-github-link-from-codebase-to-strapi-configuration
Aug 28, 2026
Merged

feat: add Github URL to navbar single type#98
matej-marcisovsky merged 1 commit into
mainfrom
75-change-github-link-from-codebase-to-strapi-configuration

Conversation

@matej-marcisovsky

@matej-marcisovsky matej-marcisovsky commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

Task Link

#75

Description

  • Make the GitHub star button URL CMS-configurable instead of hardcoded.
  • Add githubUrl on the Navbar component (next to githubStars), default https://github.com/strapi/strapi, visible only when stars are enabled.
  • Use that URL for both the button href and the GitHub stars API lookup.

Checklist - author

  • Assignee is set (the active developer of this PR).
  • Reviewers are set (team members who will review the code).
  • PR is linked I have linked the PR to an issue in Jira.
  • Description I have described the changes proposed in the PR clearly and concisely (more important than what is why).
  • Screenshots/Videos I have added screenshots/videos or they were not needed.
  • Environmental variables are listed or they were not needed.
  • Unit/Integration tests are added for all utilities and business important logic
  • Labels I have added labels to the PR.
    • Use requires-env-vars when PR requires environmental variables
    • Use ready-for-review when PR is ready to be looked at
    • Use ready-to-merge when PR has a passing review
    • Use needs-revisions when PR has comments that block merge
    • Use do-not-review when PR is daft only and you don't want to get a review
  • [OPTIONAL] Conversation/documentation is started in places where the code is not self-explanatory

Before merging

  • PR is approved
  • All conversations are resolved
  • Pipelines are not failing

Checklist - reviewer

  • I understand all proposed changes.
  • Necessary after deploy steps are handled by someone (adding new env var, syncing config, running migration...)

@vercel

vercel Bot commented Aug 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
website-ui Ready Ready Preview Aug 23, 2026 1:30pm

Request Review

@matej-marcisovsky matej-marcisovsky self-assigned this Aug 23, 2026
@matej-marcisovsky

Copy link
Copy Markdown
Collaborator Author

You can not test this in Vercel preview (no Strapi deployed).

@matej-marcisovsky
matej-marcisovsky merged commit 39d8c3f into main Aug 28, 2026
4 checks passed
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.

1 participant