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

Add sagas for fetching from backend #74

Closed
remo5000 opened this issue May 31, 2018 · 0 comments
Closed

Add sagas for fetching from backend #74

remo5000 opened this issue May 31, 2018 · 0 comments
Labels
Enhancement New feature request

Comments

@remo5000
Copy link
Contributor

No description provided.

@remo5000 remo5000 added the Enhancement New feature request label May 31, 2018
@remo5000 remo5000 added this to To do in Sprint 2 via automation May 31, 2018
@remo5000 remo5000 removed this from To do in Sprint 2 Jun 7, 2018
@remo5000 remo5000 mentioned this issue Jun 18, 2018
Merged
8 tasks
@ning-y ning-y closed this as completed in b388c03 Jun 20, 2018
Aulud pushed a commit to Aulud/cadet-frontend that referenced this issue May 25, 2020
@ning-y @indocomsoft
Implement Updater as a GenServer (source-academy#74)
b31f738 
* Add Updater.get_course_id/1

* Add test for Updater.get_course_id/1

* Add tests for IVLE.api_fetch/2

* Make Updater implement GenServer

* Fix invalid token failing silently (announcements)

* Add Updater to the supervisor tree

* Rename IVLE's api_fetch/2 -> api_call/2

* Update documentation

* Add test for new guarded IVLE.api_call/2

* Add test for Updater.get_announcements#1

* Add tests for Updater.init/1

* Add tests for Updater.handle_info/2

* Fix missing cassette in updater_test

* Refactor a line with pipe operators

* Improve test coverage for Updater

* Fix private function being public

* Bump test coverage for Updater to 100%

* Document interval being milliseconds with comments

* Refactor updater

* Remove redundant case block

* Replace Enum.filter with Enum.find

* Refactor tests

* Add test for get_api_params/0

* Fix reduced test coverage

* Add command line flag to toggle updater
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement New feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@remo5000