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(core): added event source polyfill in develop to fix IE/edge hmr #11582

Merged
merged 2 commits into from
Feb 13, 2019

Conversation

saranshkataria
Copy link
Contributor

fixes #11495

Description

Added event source polyfill in develop mode for IE. I am not sure how to test the CLI after making these changes, would need some pointers for that.

@saranshkataria saranshkataria requested a review from a team as a code owner February 6, 2019 02:52
Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

Fixed the lint changes, so this looks good.

@saranshkataria were you able to validate this solves the issue for Internet Explorer/Windows? If you need any assistance with that, I'd recommend checking out gatsby-dev-cli, which lets you link a local version of Gatsby within a project.

@saranshkataria
Copy link
Contributor Author

I tried using gatsby-dev-cli, but was not able to, but do not see the event source plugin hooked up when I run it. I am not sure if I am missing something while setting up gatsby-dev-cli or I missed some statement while importing the polyfill. I looked for some implementations in other projects, they have added the same files that I added, but I cannot get it working for me somehow. Would need some help on how to proceed further.

@pieh
Copy link
Contributor

pieh commented Feb 7, 2019

Yeah, gatsby-dev-cli doesn't support yet installing added/modified deps - you would need to manually install event-source-polyfill first in your example site, and then run gatsby-dev

@saranshkataria
Copy link
Contributor Author

@pieh already did that, but still, IE throws the warning that event source is missing

@pieh
Copy link
Contributor

pieh commented Feb 7, 2019

I will take a look at it tomorrow

@saranshkataria
Copy link
Contributor Author

@pieh did you get a chance to look into it?

@pieh pieh self-assigned this Feb 13, 2019
@pieh
Copy link
Contributor

pieh commented Feb 13, 2019

I did not - I keep doing too many TO-DOs to complete them all :(

@wardpeet
Copy link
Contributor

@pieh I'll check this one :)

@pieh pieh assigned wardpeet and unassigned pieh Feb 13, 2019
Copy link
Contributor

@wardpeet wardpeet left a comment

Choose a reason for hiding this comment

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

Sweet works like a charm! Thank you for fixing this one!

@wardpeet wardpeet changed the title added event source polyfill in case of develop mode fix(core): added event source polyfill in develop to fix IE/edge hmr Feb 13, 2019
@wardpeet wardpeet merged commit 8a6db6a into gatsbyjs:master Feb 13, 2019
gurpreet-hanjra pushed a commit to gurpreet-hanjra/gatsby that referenced this pull request Feb 14, 2019
…atsbyjs#11582)

fixes gatsbyjs#11495

## Description

Added event source polyfill in develop mode for IE. I am not sure how to test the CLI after making these changes, would need some pointers for that.

Co-authored-by: Dustin Schau <DSchau@users.noreply.github.com>
m-allanson added a commit to gurpreet-hanjra/gatsby that referenced this pull request Feb 15, 2019
* master: (76 commits)
  chore(release): Publish
  Update relay version once again (gatsbyjs#11767)
  Update CHANGELOG.md (gatsbyjs#11764)
  chore(release): Publish
  Move to @gatsbyjs scoped version of yarn (gatsbyjs#11759)
  fix(blog): 2019-01-01 json code blocks (gatsbyjs#11750)
  fix(starters): update dependency gatsby to ^2.1.0 (gatsbyjs#11745)
  fix(starters): update dependency prop-types to ^15.7.2 (gatsbyjs#11748)
  feat(showcase): add Incremental.com.au (gatsbyjs#11729)
  feat(starters): add starter magicsoup.io (gatsbyjs#11670)
  docs(gatsby): Add documentation for useStaticQuery (gatsbyjs#11741)
  chore(release): Publish
  feat(gatsby): add useStaticQuery hook (gatsbyjs#11588)
  chore(release): Publish
  chore(docs): reword CSS in JS docs for clarity (gatsbyjs#11439)
  chore: Upgrade Prettier related packages to the latest (gatsbyjs#11735)
  fix(core): added event source polyfill in develop to fix IE/edge hmr (gatsbyjs#11582)
  chore: minify svg husky hook (gatsbyjs#10560)
  docs: add videos for Gatsby Link + rewrite for flow (gatsbyjs#11700)
  docs: add egghead lesson to quickstart (gatsbyjs#11699)
  ...
pragmaticpat pushed a commit to pragmaticpat/gatsby that referenced this pull request Apr 28, 2022
…atsbyjs#11582)

fixes gatsbyjs#11495

## Description

Added event source polyfill in develop mode for IE. I am not sure how to test the CLI after making these changes, would need some pointers for that.

Co-authored-by: Dustin Schau <DSchau@users.noreply.github.com>
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.

Hot module replacement not working in IE
4 participants