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.

Sign up for GitHub

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

Jump to bottom

Some problem when auto-refreshing the page by next.js dev mode. #255

Closed
dadiorchen opened this issue Nov 4, 2021 · 2 comments · Fixed by #287
Closed

Some problem when auto-refreshing the page by next.js dev mode. #255

dadiorchen opened this issue Nov 4, 2021 · 2 comments · Fixed by #287
Labels
bug Something isn't working good first issue Good for newcomers nextjs released on @beta

Comments

@dadiorchen
Copy link
Collaborator

When modifying js file, next.js will automatically reresh the web page, but sometimes it show error:
image

This is related to the map object mounting to the DOM, maybe we need a better way to mount the map object to the DOM

Some hints:

  • Please read our readme for more information/guide/tutorial.
  • Here is an engineering book in Greenstand.
  • To know more about our organization, visit our website.
  • If you want to join the slack community (some resources need the community member's permission), please leave your email address.
@dadiorchen dadiorchen added bug Something isn't working good first issue Good for newcomers nextjs labels Nov 4, 2021
@LahiaOmar
Copy link
Contributor

Hey @dadiorchen, I would like to start working/fixing this bug

LahiaOmar added a commit to LahiaOmar/treetracker-web-map-client that referenced this issue Nov 4, 2021
@LahiaOmar
fix: auto-refreshing the page in dev mode Greenstand#255
a626d58
dadiorchen added a commit that referenced this issue Nov 8, 2021
@dadiorchen
Merge pull request #262 from LahiaOmar/web-map-site
c1d8131 
fix: auto-refreshing the page in dev mode #255
@QuaidBartolomei QuaidBartolomei mentioned this issue Nov 19, 2021
Merged
11 tasks
@github-actions
Copy link

🎉 This issue has been resolved in version 1.15.0-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers nextjs released on @beta
Projects
No open projects
Web Map Application
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Material-UI 5, React 17, Nextjs 11 migration QuaidBartolomei/treetracker-web-map-client
2 participants
@dadiorchen @LahiaOmar