Skip to content

Prefer structured clone#319

Merged
JackWilb merged 2 commits intomainfrom
prefer-structuredClone
Jul 7, 2022
Merged

Prefer structured clone#319
JackWilb merged 2 commits intomainfrom
prefer-structuredClone

Conversation

@JackWilb
Copy link
Copy Markdown
Member

@JackWilb JackWilb commented Jul 7, 2022

Does this PR close any open issues?

Closes #138

Give a longer description of what this PR addresses and why it's needed

Updates dependencies to enable a new eslint version with globals that will allow us to use structredClone instead of JSON.parse(JSON.stringify(...)).

Provide pictures/videos of the behavior before and after these changes (optional)

N/A

Are there any additional TODOs before this PR is ready to go?

TODOs:

  • Check that the previews/production deploys work as intended

@netlify
Copy link
Copy Markdown

netlify bot commented Jul 7, 2022

Deploy Preview for next-multilink ready!

Name Link
🔨 Latest commit a38c6d7
🔍 Latest deploy log https://app.netlify.com/sites/next-multilink/deploys/62c74d7c45e2310008710636
😎 Deploy Preview https://deploy-preview-319--next-multilink.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@JackWilb JackWilb marked this pull request as ready for review July 7, 2022 21:42
@JackWilb JackWilb merged commit ced9b3c into main Jul 7, 2022
@JackWilb JackWilb deleted the prefer-structuredClone branch July 7, 2022 21:42
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.

See if there are performance issue from JSON.parse(JSON.stringify()) for simulationLinks

1 participant