Skip to content
This repository has been archived by the owner on Feb 28, 2022. It is now read-only.

have clear error messages in console when expected column is missing #284

Open
galiat opened this issue May 21, 2019 · 2 comments
Open

have clear error messages in console when expected column is missing #284

galiat opened this issue May 21, 2019 · 2 comments

Comments

@galiat
Copy link
Collaborator

galiat commented May 21, 2019

As we try try to support other group's datasets we want to make it as easy as possible to troubleshoot why their sheet isn't working with our code.

Add console statements for any important column names that are missing. If you need help determining what is an important column name, ask in the slack channel.

We may in the future make this even more user friendly with a helpful real UI but for now lets stick to what is most simple.

@galiat galiat changed the title Clear error messages in console when expected column is missing have clear error messages in console when expected column is missing May 21, 2019
@galiat galiat added the CfB - good first issue Good for newcomers label May 22, 2019
@jamesgburke jamesgburke self-assigned this Jun 12, 2019
@acharliekelly
Copy link
Collaborator

There should be console messages for debugging, but also some kind of user feedback beyond the infinite progress circle, so the user knows that the app is broken and they should contact whoever the admin is and let them know to stop changing column names

@acharliekelly
Copy link
Collaborator

BTW, for whomever is working on this issue: Tabletop has a 'debug' property that will give verbose messages about what it's doing. Could probably help with figuring out what's up with bad column names

@galiat galiat removed the CfB - good first issue Good for newcomers label Oct 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants