Dashboard ids hashing leads to unexpected UX

quantified-uncertainty / metaforecast

Fetch forecasts from prediction markets/forecasting platforms to make them searchable. Integrate these forecasts into other services.

https://metaforecast.org/

MIT License

56 stars 5 forks source link

Dashboard ids hashing leads to unexpected UX #61

Open berekuk opened 2 years ago

berekuk commented 2 years ago

If I go to https://metaforecast.org/dashboards and create a dashboard with example ids but a modified title, I'll get someone else's dashboard (e.g., the default one), because INSERT in pgInsertIntoDashboard fails quietly and dashboard id is built from question ids only.

Is this a feature or a bug? Seems more like a bug to me, but I'm not entirely sure.

Maybe we should switch to the random uuids/cuids instead of relying on hashing?

NunoSempere commented 2 years ago

Yeah, this was a badly thought out feature, is fine to change. I'd have a weak preference for SHA-hashing the whole object, rather than just the array of ids