Skip to content

Remove Dashboard link#8

Merged
ImFelixprogram merged 2 commits intoEasy-Systems:mainfrom
Warsheep-GER:dashboard
Sep 24, 2024
Merged

Remove Dashboard link#8
ImFelixprogram merged 2 commits intoEasy-Systems:mainfrom
Warsheep-GER:dashboard

Conversation

@Warsheep-GER
Copy link
Contributor

-removed dashboard link from footer bec. its not for public

Pre-flight checklist

  • I have read the Contributing Guidelines on pull requests.
  • If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
  • If this is a substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

Test Plan

image

Test links

https://github.com/Warsheep-GER/documentations/blob/dashboard/docusaurus/docusaurus.config.ts

Related issues/PRs

#5

-removed dashboard link from footer
@TheMelone2 TheMelone2 self-assigned this Sep 24, 2024
@TheMelone2 TheMelone2 self-requested a review September 24, 2024 17:47
Copy link
Contributor

@TheMelone2 TheMelone2 left a comment

Choose a reason for hiding this comment

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

Valid change.

@ImFelixprogram
Copy link
Contributor

I saw this coming. That pr should've been merged first since the old app invite link is still in there

@ImFelixprogram ImFelixprogram merged commit 171a6a9 into Easy-Systems:main Sep 24, 2024
@Warsheep-GER Warsheep-GER deleted the dashboard branch September 24, 2024 18:57
@TheMelone2 TheMelone2 added the Ready for production This PR / Code is ready for production label Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready for production This PR / Code is ready for production

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants