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

Add re-login information after successful action #21

Merged
merged 1 commit into from
Feb 22, 2019

Conversation

sharidas
Copy link
Contributor

@sharidas sharidas commented Dec 1, 2017

After the command is executed successfully by
recreating masterkey, the user should be informed
to relogin.

Signed-off-by: Sujith H [email protected]

@sharidas sharidas self-assigned this Dec 1, 2017
@sharidas sharidas added this to the QA milestone Dec 1, 2017
@sharidas sharidas force-pushed the relogin-info-recreate-masterkey branch from 7fbe5b8 to aa42143 Compare December 1, 2017 11:28
@codecov-io
Copy link

codecov-io commented Dec 1, 2017

Codecov Report

Merging #21 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #21      +/-   ##
============================================
+ Coverage     64.26%   64.28%   +0.01%     
  Complexity      598      598              
============================================
  Files            33       33              
  Lines          2211     2212       +1     
============================================
+ Hits           1421     1422       +1     
  Misses          790      790
Impacted Files Coverage Δ Complexity Δ
lib/Command/RecreateMasterKey.php 86.07% <100%> (+0.17%) 10 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09b2b51...326a738. Read the comment docs.

Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

see comment, let's do it properly and get rid of both globals

@PVince81 PVince81 assigned sharidasan and unassigned sharidas and sharidasan Oct 23, 2018
@PVince81 PVince81 modified the milestones: QA, backlog Oct 23, 2018
@sharidas sharidas force-pushed the relogin-info-recreate-masterkey branch from aa42143 to 9c16d33 Compare February 21, 2019 08:38
@sharidas
Copy link
Contributor Author

The imho CI failure doesn't look related to this change...

After successful re-creation of masterkey, its always
better to let user know that one needs to re-login.

Signed-off-by: Sujith H <[email protected]>
@sharidas sharidas force-pushed the relogin-info-recreate-masterkey branch from a0fe6b5 to 326a738 Compare February 22, 2019 06:09
Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@PVince81 PVince81 merged commit 0ce6b6d into master Feb 22, 2019
@delete-merged-branch delete-merged-branch bot deleted the relogin-info-recreate-masterkey branch February 22, 2019 15:01
@PVince81
Copy link
Contributor

@sharidas please backport

@sharidas
Copy link
Contributor Author

Backport to stable10 : owncloud/core#34596

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

Successfully merging this pull request may close these issues.

5 participants