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.

Sign up for GitHub

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

Jump to bottom

DB limitations and new routes #37

Merged
merged 5 commits into from
Oct 8, 2020
Merged

DB limitations and new routes #37

merged 5 commits into from
Oct 8, 2020

Conversation

InbarShirizly
Copy link
Owner

@InbarShirizly InbarShirizly commented Oct 8, 2020 •
edited
Loading

  • limitations:

    • Students_list in excel file can have more the X students (currently 200)
    • student list can't be empty
    • chat file must have the format of a classic zoom chat file downloaded from the zoom app

  • update postman API documentation - update API-doc (postman) #36

API-doc

image

image

InbarShirizly and others added 5 commits October 8, 2020 10:12
@InbarShirizly
Change test folder to tests
eeba053
@InbarShirizly
valdations of uploading files in validator class +
36272c8 
configuration of parsers moved to config, report date is only user input (regardless the chat time).
@InbarShirizly
#24 move storing chat related data - no support for chat
267a08f 
currently decision to not support chat storage and limitation on it - created a config for it
@ItayS14
Added endpoints to get and update student data
0338a8e
@InbarShirizly
report will return description
4fec61d
@InbarShirizly InbarShirizly added the enhancement New feature or request label Oct 8, 2020
@InbarShirizly InbarShirizly merged commit 1241ad1 into master Oct 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Allow teacher to change student name in the database
2 participants
@InbarShirizly @ItayS14