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

feat: V2.0 #493

Merged
merged 14 commits into from
Sep 23, 2023
Merged

feat: V2.0 #493

merged 14 commits into from
Sep 23, 2023

Conversation

yihong0618
Copy link
Owner

@yihong0618 yihong0618 commented Sep 17, 2023

  1. typescript
  2. vite build
  3. remove gatsby
  4. change scripts dir name to run_page for future develop
  5. update to react 18 and react-map-gl version
  6. better actions
  7. use pnpm

yihong0618 and others added 9 commits October 8, 2022 14:46
* feat: Typescript!

Signed-off-by: Frost Ming <[email protected]>

* chore: use node 16 as the base

Signed-off-by: Frost Ming <[email protected]>

* fix: remove use dependencies

Signed-off-by: Frost Ming <[email protected]>

---------

Signed-off-by: Frost Ming <[email protected]>
feat: change to react and vite

Signed-off-by: Frost Ming <[email protected]>
* This commit do these things,
cause react update to 18 there is a serious bug for
react-map-gl, so we deside to update it to v7.
for v7 there's a lot of break change, we fix them as follows:
- bring mapbox css to the file.
- change the api that break
- add two packages in it to fit the change
- change viewport name to viewState to fit the api change

* fix: git ignore

* style:
* line blur to smooth
* mapStyle using dark-v10 to solve the language problem
* FullscreenButton style fix

---------

Co-authored-by: ben_29 <[email protected]>
@vercel
Copy link

vercel bot commented Sep 17, 2023

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

Name Status Preview Comments Updated (UTC)
running-page ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 18, 2023 1:03pm

Copy link
Collaborator

@ben-29 ben-29 left a comment

Choose a reason for hiding this comment

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

Great jobs, I've tested it in local dev env, works like a charm.
One small issue, I think you need to reimport the project on Vercel and start as Vite project, or the preview link won't work.

@yihong0618
Copy link
Owner Author

@ben-29 yes...

@yihong0618 yihong0618 merged commit 19bfd3c into master Sep 23, 2023
7 checks passed
@yihong0618 yihong0618 deleted the v2.0 branch September 23, 2023 10:00
@yihong0618 yihong0618 restored the v2.0 branch September 23, 2023 10:01
@yihong0618 yihong0618 temporarily deployed to github-pages September 23, 2023 10:02 — with GitHub Actions Inactive
@frostming frostming deleted the v2.0 branch September 25, 2023 06:44
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.

5 participants