Skip to content
This repository has been archived by the owner on Jul 30, 2024. It is now read-only.

Fixes ActivateUser and DeactivateUser commands #503

Closed
wants to merge 1 commit into from
Closed

Fixes ActivateUser and DeactivateUser commands #503

wants to merge 1 commit into from

Conversation

edouardberthe
Copy link

Converts user_identifier coming from ActivateUserCommand and DeactivateUserCommand to user entity in datastore.

@edouardberthe
Copy link
Author

#458

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 94.229% when pulling 9706447bfbb9a10501df826965c6b5b49a373dea on edouardberthe:develop into 301fc03 on mattupstate:develop.

…teUserCommand to user entity in datastore + add test
@edouardberthe
Copy link
Author

Hi, could I please have a review on this PR?
I have added a test to correct the slight decrease in test coverage.
Thx

@edouardberthe
Copy link
Author

Hi, could I have please a review on this PR ?

Thanks

@jirikuncar
Copy link
Contributor

@edouardberthe wouldn't be better to fix it in the script.py:_ToggleActiveCommand?

@jirikuncar
Copy link
Contributor

New Click command line interface has been implemented in 93ae74b.

@jirikuncar jirikuncar closed this Apr 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants