Skip to content

Add request parameter to delete user method - #1258

Merged
frankie567 merged 2 commits into
fastapi-users:masterfrom
hgalytoby:add-request-parameter-to-delete-user-method
Jul 26, 2023
Merged

frankie567 merged 2 commits into
fastapi-users:masterfrom
hgalytoby:add-request-parameter-to-delete-user-method

Conversation

@hgalytoby

Copy link
Copy Markdown
Contributor

#1257

  • add request parameter to delete_user method

  • Docs: Add request parameter to docstring

@codecov

codecov Bot commented Jul 25, 2023 •

Copy link
Copy Markdown

Codecov Report

Merging #1258 (d4cb20b) into master (cd325da) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master     #1258   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           35        35           
  Lines          969       969           
=========================================
  Hits           969       969           
Files Changed Coverage Δ
fastapi_users/manager.py 100.00% <ø> (ø)
fastapi_users/router/users.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@frankie567

Copy link
Copy Markdown
Member

Nothing to add, that's perfect! Thanks 🙏

@all-contributors add @hgalytoby for bug

@frankie567
frankie567 merged commit fe932fe into fastapi-users:master Jul 26, 2023
@allcontributors

Copy link
Copy Markdown
Contributor

@frankie567

I've put up a pull request to add @hgalytoby! 🎉

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