bundly / dash

From automated standup notes to GitHub notifications a single haven for your fellowship needs
https://dash.bundly.tech
MIT License
10 stars 10 forks source link

Static user data fetch script and dump #6

Closed darshkpatel closed 4 years ago

darshkpatel commented 4 years ago
vercel[bot] commented 4 years ago

This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/bundly/dash/8k4gq6tp3 ✅ Preview: https://dash-git-feat-user-data.bundly.vercel.app

sauravhiremath commented 4 years ago
darshkpatel commented 4 years ago
  • So are we keeping this as a script only? (Can be run only when there is an update)

Yep, Users and their pods will remain static throughout the fellowship, in case the need arises the script can be used to update the users

  • Also, do we keep this fetched data in memory (as JSON in repo) or store it in a DB?

small list, can be kept in memory itself, plus we aren't running any complex queries on these either

darshkpatel commented 4 years ago

Could you run a sanity check on json with usernames as key ( to ensure there are no repeated values). You have already excluded parent teams, and it should work fine for now. But if in future any other nested teams are added, then sanity check could work) - Can just loop over the list to remove duplicates as the last step

Was a bigger problem , added support for people with multiple pods!

darshkpatel commented 4 years ago

Yeah LGTM. Can run the scripts again to update it

Already done in cc33418