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

findByIdAndRemove depricated? #30152

Closed
Shahad7 opened this issue Nov 9, 2023 · 4 comments · Fixed by #30230
Closed

findByIdAndRemove depricated? #30152

Shahad7 opened this issue Nov 9, 2023 · 4 comments · Fixed by #30230
Assignees
Labels
Content:Learn:Express Learning area Express docs

Comments

@Shahad7
Copy link

Shahad7 commented Nov 9, 2023

MDN URL

https://developer.mozilla.org/en-US/docs/Learn/Server-side/Express_Nodejs/forms/Delete_author_form

What specific section or headline is this issue about?

delete author post route

What information was incorrect, unhelpful, or incomplete?

I believe the findByIdAndRemove function doesn't exist anymore. It was throwing an error and I couldn't find the function in the mongoosejs ofiicial docs. I was following the tutorial

What did you expect to see?

a function that deletes the record ofc!

Do you have any supporting links, references, or citations?

No response

Do you have anything more you want to share?

No response

MDN metadata

Page report details
@Shahad7 Shahad7 added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Nov 9, 2023
@github-actions github-actions bot added the Content:Learn:Express Learning area Express docs label Nov 9, 2023
@hamishwillee hamishwillee self-assigned this Nov 13, 2023
@hamishwillee hamishwillee removed the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Nov 13, 2023
@hamishwillee
Copy link
Collaborator

Thanks very much - this should be fixed by #30230

@Shahad7
Copy link
Author

Shahad7 commented Nov 13, 2023 via email

@hamishwillee
Copy link
Collaborator

hamishwillee commented Nov 13, 2023

Hi @Shahad7 No need thanks. If you look at the issue I created it has Fixes #30152 in the description. What that means is that as soon as the PR is merged this issue will automatically close.

We do it that way because sometimes a reviewer will reject a PR fix as "not good enough" - in which case this issue would remain open until it was actually fixed.

@Shahad7
Copy link
Author

Shahad7 commented Nov 13, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Learn:Express Learning area Express docs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants