search

r4ds / shinyslack

Integrate Slack and Shiny
Other
15 stars 3 forks source link

Error handling for user_info #48

Closed jonthegeek closed 6 months ago

jonthegeek commented 1 year ago

If the API call that hits "users.info" fails, we should be noisier about it. I strongly suspect we should delete the cookie and have them re-validate, but I need to think about whether there can be another cause.

Symptom: Lydia Gibson's display name didn't load, so it's blank in the spreadsheet. Her user_id is "U02ATHQMQ4Q", see if maybe she doesn't HAVE the field I'm looking at.

Probably coalesce if those can be blank. Dig deeper to understand.

jonthegeek commented 6 months ago

This is a usage issue, not an error in how things are loaded from Slack. When I implement a module, I'll likely copy/paste the "fix" for this from bookclubber.