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

Log actions on assembly user roles #2874

Merged
merged 8 commits into from
Mar 2, 2018
Merged

Conversation

mrcasals
Copy link
Contributor

@mrcasals mrcasals commented Mar 1, 2018

🎩 What? Why?

This PR logs actions on assembly user roles: creation, update and deletion.

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG entry

📷 Screenshots (optional)

Description

@mrcasals mrcasals added this to the Release v.0.10.0 milestone Mar 1, 2018
@mrcasals mrcasals self-assigned this Mar 1, 2018
@codecov
Copy link

codecov bot commented Mar 1, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@3f3be5d). Click here to learn what that means.
The diff coverage is 98.43%.

@@            Coverage Diff            @@
##             master    #2874   +/-   ##
=========================================
  Coverage          ?   98.73%           
=========================================
  Files             ?     1649           
  Lines             ?    39127           
  Branches          ?        0           
=========================================
  Hits              ?    38631           
  Misses            ?      496           
  Partials          ?        0

@mrcasals
Copy link
Contributor Author

mrcasals commented Mar 2, 2018

Merging since the last real commit had its test suite passing. The merge commit is to update the CHANGELOG file.

@mrcasals mrcasals merged commit 14d1457 into master Mar 2, 2018
@mrcasals mrcasals deleted the log/assembly-user-roles branch March 2, 2018 08:39
@ghost ghost removed the in-review label Mar 2, 2018
@mrcasals mrcasals modified the milestones: Release v.0.10.0, CDP2 Dec 10, 2018
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