Skip to content

[encryption] some visual improvements#11453

Merged
MorrisJobke merged 4 commits intomasterfrom
enc_visual_improvements
Oct 8, 2014
Merged

[encryption] some visual improvements#11453
MorrisJobke merged 4 commits intomasterfrom
enc_visual_improvements

Conversation

@schiessle
Copy link
Copy Markdown
Contributor

Improved visual feedback as described here #11393

Additional improved feedback if the user enabled/disables the recovery key. Now it uses the same read/green tags like everywhere else.

@scrutinizer-notifier
Copy link
Copy Markdown

The inspection completed: 5 new issues, 2 updated code elements

@ghost
Copy link
Copy Markdown

ghost commented Oct 7, 2014

🚀 Test PASSed. 🚀
Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng/130/:rocket: Test PASSed. 🚀

@schiessle
Copy link
Copy Markdown
Contributor Author

Some small visual improvements, @ser72 already tested it here: #11393 (comment)

Who wants to be the second reviewer? Maybe @PVince81 @MorrisJobke @th3fallen ? Thanks!

@MorrisJobke
Copy link
Copy Markdown
Contributor

I'm just reviewing :)

@schiessle
Copy link
Copy Markdown
Contributor Author

Great, Thanks a lot!

@MorrisJobke
Copy link
Copy Markdown
Contributor

Works 👍

MorrisJobke added a commit that referenced this pull request Oct 8, 2014
[encryption] some visual improvements
@MorrisJobke MorrisJobke merged commit 6d94a88 into master Oct 8, 2014
@MorrisJobke MorrisJobke deleted the enc_visual_improvements branch October 8, 2014 12:36
@MorrisJobke
Copy link
Copy Markdown
Contributor

@schiesbn @karlitschek Backport?

@schiessle
Copy link
Copy Markdown
Contributor Author

Looking at the initial issue I assume yes... I will take care of it.

@schiessle
Copy link
Copy Markdown
Contributor Author

stable7: a045f8c...a766f5b

@lock lock bot locked as resolved and limited conversation to collaborators Aug 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants