multinet-app / multilink

A node-link visualization built to work with the multinet ecosystem.
https://multinet.app
Apache License 2.0
4 stars 2 forks source link

Resolve all issues with new Set() #324

Closed JackWilb closed 2 years ago

JackWilb commented 2 years ago

Does this PR close any open issues?

Closes #127

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

Removes new Set() where possible, and we're sure now that there are no large performance issues in the areas that it's used

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?

netlify[bot] commented 2 years ago

Deploy Preview for next-multilink ready!

Name Link
Latest commit 312081952930db0dd4977d5f6c1ab52e8910903e
Latest deploy log https://app.netlify.com/sites/next-multilink/deploys/62e435454925790008f86f6d
Deploy Preview https://deploy-preview-324--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.