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

Fix update of User roles table after a role has been removed #2630

Merged
merged 4 commits into from
Jul 4, 2018

Conversation

irfanhabib
Copy link
Contributor

@irfanhabib irfanhabib commented Jul 4, 2018

Based on #2620

@irfanhabib irfanhabib self-assigned this Jul 4, 2018
@irfanhabib irfanhabib requested a review from KlapTrap July 4, 2018 12:18
@cfdreddbot
Copy link

Hey irfanhabib!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@codecov
Copy link

codecov bot commented Jul 4, 2018

Codecov Report

Merging #2630 into v2-master will increase coverage by <.01%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##           v2-master    #2630      +/-   ##
=============================================
+ Coverage      69.92%   69.93%   +<.01%     
=============================================
  Files            596      596              
  Lines          25289    25291       +2     
  Branches        5716     5716              
=============================================
+ Hits           17684    17686       +2     
  Misses          7605     7605

@KlapTrap KlapTrap merged commit c484b2a into v2-master Jul 4, 2018
@nwmac nwmac deleted the fix-2617 branch June 12, 2019 18:49
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.

3 participants