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

No mention of "sort_by" argument of get_grades() in README #14

Closed
maciejkrol18 opened this issue Apr 17, 2023 · 1 comment
Closed

No mention of "sort_by" argument of get_grades() in README #14

maciejkrol18 opened this issue Apr 17, 2023 · 1 comment

Comments

@maciejkrol18
Copy link
Contributor

In the README, the usage of get_grades is shown as follows:

from librus_apix.grades import get_grades

grades, average_grades = get_grades(token)

for mark in grades["Mathematics"]:
    print(mark.grade)

Running this code results in TypeError: get_grades() missing 1 required positional argument: 'sort_by'

@RustySnek
Copy link
Owner

The sort_by argument is for getting new grades or grades since last login. I actually forgot about it when writing the README. I will set the sort_by to get all grades by default and update the README file.

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

2 participants