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

Use number for sessionId and handle states that are objects (not string) #380

Closed JackWilb closed 1 year ago

JackWilb commented 1 year ago

Does this PR close any open issues?

No

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

There was a bug that allowed us to save the session as a string in the multinet db. This PR fixes that issue and adds an enhancement from the trrack library that allows us to import and export the provenance as an object. This simplifies some code and means we don't need to JSON dump and stringily as the states move through the multinet ecosystem.

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:

netlify[bot] commented 1 year ago

Deploy Preview for next-multilink ready!

Name Link
Latest commit 8e781bdf6af382c2ace67ef3b11f3e9710277f70
Latest deploy log https://app.netlify.com/sites/next-multilink/deploys/64d57e04e30f9d00089bbca7
Deploy Preview https://deploy-preview-380--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 configuration.