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

Changelog #13

Closed
wants to merge 1 commit into from
Closed

Changelog #13

wants to merge 1 commit into from

Conversation

chirayudesai
Copy link
Contributor

This includes the changes from #12 to make things easier.

Changelog is HTML, example can be seen at #9 (comment)

It's not exactly the best system, and the changelog downlaod only happens alongwith update download, but I feel that this is sufficiently usable.

Closes #9

@chirayudesai
Copy link
Contributor Author

I included the changes from #12 since that was easier, and this is bound to conflict with that otherwise.
Can update either changes as needed.

The changelog would be made "prettier", and I still need to figure out wrapping, but that's mostly going to be either in the HTML or maybe (unlikely) HTMLViewer.

* Changelog is HTML, shown via HTMLViewer, which is also used by
  Settings -> About -> Legal
* Changelog file: <device>-<channel>-changelog.html
* Show changelog if it exists upon tapping the notification, or
  the settings entry.

Potential TODOs:
* The view can be zoomed in and out (which may be useful),
  and also scrolled around easily (not so useful).
}

static Intent getChangelogIntent(Context context, String filename) {
// From com.android.settings.SettingsLicenseActivity.showHtmlFromUri()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This code is Apache2

@thestinger thestinger closed this Apr 15, 2020
@thestinger
Copy link
Member

By the way this got closed from removing the pie branch, not because I explicitly closed it.

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.

Show a changelog
2 participants