Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

added custom courses and provider for it, also implimented favorites … #158

Merged
merged 4 commits into from
Jun 16, 2018
Merged

added custom courses and provider for it, also implimented favorites … #158

merged 4 commits into from
Jun 16, 2018

Conversation

VoitreX
Copy link
Contributor

@VoitreX VoitreX commented Jun 16, 2018

…in the SQLite

@coveralls
Copy link

coveralls commented Jun 16, 2018 •
edited
Loading

Pull Request Test Coverage Report for Build 709

  • 0 of 21 (0.0%) changed or added relevant lines in 4 files are covered.
  • 4 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.4%) to 36.053%
Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/connect/dataprovider/user/mock/sqlite_user_settings_provider.dart 0 2 0.0%
lib/controller/session.dart 0 3 0.0%
lib/connect/dataprovider/databasehelper/databasehelper.dart 0 8 0.0%
lib/connect/dataprovider/course/sqlite_course_provider.dart 0 8 0.0%
Files with Coverage Reduction New Missed Lines %
lib/connect/dataprovider/databasehelper/databasehelper.dart 2 0.0%
lib/ui/screens/timetable_screen.dart 2 75.0%
Totals Coverage Status
Change from base Build 690: -0.4%
Covered Lines: 676
Relevant Lines: 1875
💛 - Coveralls

Gleydar
Gleydar approved these changes Jun 16, 2018
View reviewed changes
Copy link

@Gleydar Gleydar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems good

@Gleydar Gleydar merged commit 13e0692 into sprint3 Jun 16, 2018
@Gleydar Gleydar deleted the customCourses branch June 16, 2018 18:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Gleydar Gleydar Gleydar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@VoitreX @coveralls @Gleydar