Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add link to forums to navigation elements #1167

Merged
merged 1 commit into from
Aug 10, 2024

Conversation

UvixCreative
Copy link
Contributor


name: Pull request
about: Create a pull request
title: 'Add link to forums to navigation elements'
labels: ''
assignees: ''


What type of PR is this?(check all applicable)

  • refactor
  • feature
  • bug fix
  • documentation
  • optimization
  • other

Description

Added a link to the forums to navigation elements:

  • Footer
  • Unauthenticated desktop header
  • "Profile" dropdown element

Related Issues

Issue # N/A

What this PR achieves

The main website will now point users to the forums in a few locations. In the unauthenticated desktop header, the Discord link has been replaced with a link to the forums. I believe the forums would be a better place to highlight for users that are looking for community. The Discord invite link is still very accessible via the footer and the "profile" dropdown element, it has just been de-prioritized.

Screenshots, recordings

N/A

Notes

I did not build the project to test the changes, primarily because I expect them to be purely cosmetic

Copy link

vercel bot commented Aug 10, 2024

@UvixCreative is attempting to deploy a commit to the openbeta-dev Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Aug 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
open-tacos ❌ Failed (Inspect) Aug 10, 2024 5:24pm

@vnugent vnugent merged commit dcd0f12 into OpenBeta:develop Aug 10, 2024
3 of 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.

2 participants