Skip to content
This repository has been archived by the owner on Sep 19, 2022. It is now read-only.

POC Remove NEXT #1178

Draft
wants to merge 3 commits into
base: dev
Choose a base branch
from
Draft

POC Remove NEXT #1178

wants to merge 3 commits into from

Conversation

noisekit
Copy link
Contributor

@noisekit noisekit commented Jul 14, 2022 •
edited
Loading

  • Create a clean webpack + swc build
  • Add single JS entry
  • Basic HTML template
  • Temporarily alias next/head to react-helmet
  • Temporarily alias next/router to custom stub
  • Temporarily alias next/link to custom stub
  • Replace next/router and next/link with react-router

@noisekit noisekit requested a review from a team July 14, 2022 05:00
@noisekit noisekit self-assigned this Jul 14, 2022
@vercel
Copy link

vercel bot commented Jul 14, 2022 •
edited
Loading

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

Name Status Preview Updated
staking ❌ Failed (Inspect) Jul 19, 2022 at 2:45AM (UTC)

github-advanced-security[bot]
github-advanced-security bot found potential problems Jul 14, 2022
View reviewed changes
no-next/next-link.tsx Outdated
@@ -0,0 +1,4 @@
export default function Link({ ...props }) {
// eslint-disable-next-line jsx-a11y/anchor-has-content
return <a {...props} />;

Check warning

Code scanning / ESLint

Enforce all anchors to contain accessible content.

Anchors must have content and the content must be accessible by a screen reader.
@noisekit noisekit changed the title WIP POC Remove NEXT POC Remove NEXT Jul 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@noisekit noisekit

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@noisekit