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

View articles offline if available #81

Merged
merged 10 commits into from
Mar 19, 2019

Conversation

thomastanck
Copy link

@thomastanck thomastanck commented Mar 18, 2019

Resolves #52 and #54.
Resolves part of #19.

Adds offlineLink field to Entry, and BrowserPanel views offlineLink if it exists.

Summary of changes

  • Add offlineLink to Entry
  • Connect offlineLink to BrowserPanel
  • Update JsonAdaptedEntry so that offlineLink will be saved to disk
  • Adds .html extension to local filenames so that WebEngine views them properly.

@thomastanck thomastanck force-pushed the offline-view branch 2 times, most recently from 6ee0aa0 to 72a5761 Compare March 18, 2019 14:27
@thomastanck thomastanck added this to the v1.2 milestone Mar 18, 2019
@coveralls
Copy link

coveralls commented Mar 18, 2019

Pull Request Test Coverage Report for Build 328

  • 51 of 54 (94.44%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.06%) to 92.602%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/main/java/seedu/address/logic/commands/AddCommand.java 3 4 75.0%
src/main/java/seedu/address/model/entry/Entry.java 15 17 88.24%
Totals Coverage Status
Change from base Build 325: -0.06%
Covered Lines: 1765
Relevant Lines: 1906

💛 - Coveralls

@thomastanck thomastanck force-pushed the offline-view branch 2 times, most recently from df22bfc to 26501b6 Compare March 18, 2019 17:09
Copy link

@rlrh rlrh left a comment

Choose a reason for hiding this comment

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

LGTM

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

Successfully merging this pull request may close these issues.

4 participants