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

Fix /Close Issues #1756

Merged
merged 4 commits into from
Jun 24, 2024
Merged

Fix /Close Issues #1756

merged 4 commits into from
Jun 24, 2024

Conversation

Innkeeping
Copy link
Contributor

@Innkeeping Innkeeping commented Jun 24, 2024

Overview

Closing minor issues, as possible.

What features/fixes does this PR include?
Changed link from old deprecated wiki to new 'WTF is Ethereum' Playbook. Updated link to current Ethereum playbook (#1739)

Updated wiki and forum links on 404 page. (#1737)
Removed Discourse/forum mentions in discord-bot

Please provide the GitHub issue number
#1739
#1737
Closes #
#1739
#1737

Follow up Improvement Ideas

  • Maybe just remove the links on start page
  • Is this start page part of the current flow of the site?
  • 404 issue mentions needing design changes?
  • router on clicks are used in home and academy links which doesn't show hover overs
  • Discord-bot still used? Still has sourcecred as well..

Copy link

vercel bot commented Jun 24, 2024

@Innkeeping is attempting to deploy a commit to the MetaFamily Team on Vercel.

A member of the Team first needs to authorize it.

Fixing forum link and wiki link on 404
Reference: Issuue MetaFam#1737
@Innkeeping Innkeeping changed the title Fix /Close Issue #1739 -Removing old link from profile opening screen Fix /Close Issues Jun 24, 2024
Copy link
Member

@dysbulic dysbulic left a comment

Choose a reason for hiding this comment

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

I'm fine with this.

@dysbulic dysbulic merged commit 312d231 into MetaFam:develop Jun 24, 2024
1 check failed
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