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

Cache what can be cached #37

Open
dbosk opened this issue Dec 17, 2020 · 4 comments · May be fixed by #60
Open

Cache what can be cached #37

dbosk opened this issue Dec 17, 2020 · 4 comments · May be fixed by #60

Comments

@dbosk
Copy link
Owner

dbosk commented Dec 17, 2020

Some things can be cached, like user and assignment information. Cache this type of data.

Add an option to store that cache on disk between sessions.
@dbosk
Copy link
Owner Author

dbosk commented Sep 26, 2021

Have a similar construction as for ladok3.

This is probably easiest done by providing a wrapper for canvasapi.

@dbosk
Copy link
Owner Author

dbosk commented Sep 27, 2021

We can pull again (refresh cache) whenever we don't get a hit.

@dbosk
Copy link
Owner Author

dbosk commented Feb 28, 2023

Having a cache will prevent running parallell instances. Or at least complicating it.

@dbosk dbosk linked a pull request Sep 15, 2024 that will close this issue
Draft
@dbosk
Copy link
Owner Author

dbosk commented Sep 15, 2024

Can we use this cache also when we include canvaslms from other packages, such as nytid? See Section 5.2.3 Looking up usernames in Canvas and LADOK in the nytid documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add caching dbosk/canvaslms
1 participant
@dbosk