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

UX: clean up CLI setup and basic usage #12

Merged
merged 7 commits into from
Mar 11, 2023
Merged

UX: clean up CLI setup and basic usage #12

merged 7 commits into from
Mar 11, 2023

Conversation

zmoog
Copy link
Owner

@zmoog zmoog commented Dec 31, 2022

Motivation

The essential UX is a little chaotic. I want to design how to set up and run simple queries.

Change description

  • Remove hard-coded values, so we can run queries using filters (year, period, different students, and more).
  • Remove the 'Obiettivi' field; it looks like teachers don't use it.
  • Partial Navigator refactoring to make it work across different years and periods; in the previous version it only worked well with the current year and period.
  • Add --verbose option to print request and response details

Additional Notes

Reviewer checklist

  • PR address a single concern.
  • PR title and description are appropriately filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.
  • Docs are updated (at least the README.md, if needed).
  • History is clean, commit messages are meaningful (see CONTRIBUTING.md) and are well-formatted.

@zmoog zmoog self-assigned this Dec 31, 2022
@zmoog zmoog added enhancement New feature or request ux labels Dec 31, 2022
@zmoog zmoog changed the title Cleanup UX: clean up CLI setup and basic usage Dec 31, 2022
@zmoog zmoog marked this pull request as ready for review March 11, 2023 08:07
@zmoog zmoog merged commit ebd7826 into main Mar 11, 2023
@zmoog zmoog deleted the zmoog/cleanup branch March 11, 2023 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clean up grade list command
1 participant