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

Removed screensaver #3890

Merged
merged 1 commit into from Oct 25, 2018

Conversation

Projects
None yet
2 participants
@mdrose
Copy link
Contributor

mdrose commented Oct 19, 2018

Status

Ready. Only remaining references to the screensaver should be the in the localization. Not sure what the proper procedure is for removing those.

Description of Changes

Resolves #3722

Reverts:
f693105
f9486a6
aea2854

@redshiftzero
Copy link
Member

redshiftzero left a comment

thanks @mdrose! regarding changes to the localization files, those get updated by our localization manager, so not modifying them is the right thing to do

@redshiftzero redshiftzero merged commit 340b363 into freedomofpress:develop Oct 25, 2018

5 checks passed

ci/circleci: admin-tests Your tests passed on CircleCI!
Details
ci/circleci: lint Your tests passed on CircleCI!
Details
ci/circleci: staging-test-with-rebase Your tests passed on CircleCI!
Details
ci/circleci: tests Your tests passed on CircleCI!
Details
ci/circleci: updater-gui-tests Your tests passed on CircleCI!
Details

@mdrose mdrose deleted the mdrose:remove-screensaver branch Oct 25, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment