Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Send error messages to user if CSV uploaded incorrectly #23

Open
isha-san opened this issue Dec 2, 2021 · 2 comments
Open

Send error messages to user if CSV uploaded incorrectly #23

isha-san opened this issue Dec 2, 2021 · 2 comments
Assignees

Comments

@isha-san
Copy link
Collaborator

isha-san commented Dec 2, 2021

  • Create a separate table in Hasura that logs CSV uploads. Table will have the following columns: file name, time uploaded, formatted properly, error message school district (and whatever other columns you think would be helpful!)
  • Chain another Lambda function that triggers when our current lambda finishes (@katie-proj - put your sanitization code in the second lambda). the second lambda will check if the data is formatted properly and then based on that will update the CSV upload table with an appropriate status and error entry.
  • Add another column to the Students table, called "added_with", which keeps track of the file from the CSV upload table with which that row of data was added to the table.
  • To retrieve information about whether the data was formatted correctly in the client, make a graphql query to hasura after the file is uploaded

Assigning Bach for support :)

@isha-san
Copy link
Collaborator Author

isha-san commented Dec 4, 2021

UPDATED TASK:

  • Chain another lambda function before the sendToHasura function that will take care of data sanitization and create custom error messages for the CSV upload table
  • "Added_with" column (@BachToTheFuture and @isha-san will provide more info on this)

@isha-san
Copy link
Collaborator Author

isha-san commented Dec 4, 2021

INFO FOR @katie-proj
Ethnicity field should contain one of the following: American Indian or Alaskan Native, Asian or Asian American, Black or African American, Hawaiian Native or Other Pacific Islander, Hispanic, Other, Two or more Races, White or Caucasian
Gender field should contain Male, Female or Non-Binary
Enrollment status is Active/not Active
Will get more clarity on this from David :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants