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.

Sign up for GitHub

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

Jump to bottom

Add private_note field to User for admins #286

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Add private_note field to User for admins #286

wants to merge 1 commit into from

Conversation

yanicolivier
Copy link
Contributor

Q R
Type of contribution ? [Features]
Tickets (issues) concerned [277]
What have you changed ?

Added a private_note field on User for special note taken by the admin

Verification :

  • My branch name respect the standard defined in CONTRIBUTING.md
  • All my commits respect the standard defined in CONTRIBUTING.md
  • This Pull-Request fully meets the requirements defined in the issue
  • I added or modified the attached tests
  • I added or modified the documentation

@yanicolivier yanicolivier force-pushed the feature-private_note_user branch 2 times, most recently from 977897b to 4daf5dd Compare January 20, 2021 14:39
@coveralls
Copy link

coveralls commented Jan 20, 2021 •
edited
Loading

Coverage Status

Coverage remained the same at 0.0% when pulling 0997092 on yanicolivier:feature-private_note_user into 0ce8df0 on Volontaria:develop.

RignonNoel
RignonNoel requested changes Jan 20, 2021
View reviewed changes
api_volontaria/apps/user/tests/tests_view_users.py Outdated Show resolved Hide resolved
api_volontaria/apps/user/tests/tests_view_users.py Outdated Show resolved Hide resolved
api_volontaria/apps/user/tests/tests_view_users.py Outdated Show resolved Hide resolved
api_volontaria/apps/user/views.py Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RignonNoel RignonNoel RignonNoel requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yanicolivier @coveralls @RignonNoel