Skip to content

Conversation

rueckstiess
Copy link
Contributor

@rueckstiess rueckstiess commented May 10, 2016

Before, the "Enable Auto Updates" choice in the network opt-in panel was not persisted. Now it is. Also changed Auto to Automatic in the checkbox label and added a full stop to make it consistent with the other choices.


This change is Reviewable

@imlucas
Copy link
Contributor

imlucas commented May 10, 2016

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@imlucas
Copy link
Contributor

imlucas commented May 10, 2016

:lgtm:

Previously, rueckstiess (Thomas Rückstieß) wrote…

persist auto update choice in preferences.

Before, the "Enable Auto Updates" choice in the network opt-in panel was not persisted. Now it is. Also changed Auto to Automatic in the checkbox label and added a full stop to make it persistent with the other choices.


Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.


Comments from Reviewable

@imlucas imlucas merged commit c203465 into 1.2-releases May 10, 2016
@imlucas imlucas deleted the fix-auto-update-preferences branch May 10, 2016 18:30
imlucas added a commit that referenced this pull request May 10, 2016
persist auto update choice in preferences.
@samweaver-zz samweaver-zz mentioned this pull request May 10, 2016
imlucas added a commit that referenced this pull request May 11, 2016
* Export leafybots github api token so assets are uploaded to GitHub

* Add environment variables for download center upload

* Make auto update preference accessible in the UI

* Canonoical display names for release assets

* Merge pull request #397 from 10gen/fix-auto-update-preferences

persist auto update choice in preferences.
@kangas kangas changed the title persist auto update choice in preferences. INT-1440 persist auto update choice in preferences. May 15, 2016
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.

2 participants