Skip to content
This repository has been archived by the owner on Jan 16, 2023. It is now read-only.

Fix/next dynamic imports #420

Merged
merged 4 commits into from
Apr 9, 2019
Merged

Fix/next dynamic imports #420

merged 4 commits into from
Apr 9, 2019

Conversation

JesseE
Copy link

@JesseE JesseE commented Apr 5, 2019

Sometimes the team page breaks when page is build for the first time.some issue with dynamic imports with next vercel/next.js#3775 (comment)

  • updated next to latest version
  • fixed team page by using require instead of importing the jobsscraper

@JesseE JesseE requested a review from brunadafonseca April 5, 2019 13:47
Copy link

@brunadafonseca brunadafonseca left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@JesseE JesseE merged commit 3b630c4 into master Apr 9, 2019
@JesseE JesseE deleted the fix/next-dynamic-imports branch April 9, 2019 07:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants