MetaCell / MetaCell/geppetto-meta
GeppettoGraphVisualization requiring number as id for nodes, should be any
- Dominant language
- JavaScript
- Stars
- 6
- Forks
- 4
- PR merge metrics
- No merged PRs in 30d
Description
https://github.com/MetaCell/geppetto-meta/blob/release/v1.0.0/geppetto.js/geppetto-ui/src/graph-visualization/Graph.js#L402
I don't see any reasons for this to be a number. We usually want there instance ids, which are strings instead.
We are getting an error on console when using strings atm.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at geppetto.js/geppetto-ui/src/graph-visualization/Graph.js around line 402, where node IDs are handled. Reproduce the console error with a string instance ID and inspect the current number requirement. Done means graph nodes accept string IDs without the reported error; verify the existing graph-visualization behavior afterward.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100