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

Popup #2525

Closed
wants to merge 3 commits into from
Closed

Popup #2525

wants to merge 3 commits into from

Conversation

ftgibran
Copy link
Contributor

Description of the Change

  • Work-In-Progress: adding a popup to the legacy website that notifies the user that neo-one.io is for Neo Legacy and that N3 docs can be found at n3.neo-one.io
  • The added package (reactjs-popup) broke the React Static build process, so this PR is actually just a demonstration. This needs to be implemented without the reactjs-popup package
  • Also needed to remove some newer TypeScript language features in order to get the website to compile

Test Plan

To run the website locally in dev mode:

  • Run rush compile-website-dev
  • When that is done (is very quick) run rush run-website-dev, which takes several minutes to finish
  • Then go to localhost:3000

Alternate Designs

See notes above.

Benefits

Clarity for users on which docs they are reading.

Possible Drawbacks

Applicable Issues

#2511
#2410

@ftgibran ftgibran closed this Dec 29, 2021
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