Skip to content

delete post functionality on each post#26

Merged
Pranav016 merged 1 commit intoALPHAVIO:masterfrom
mak-ux:master
Jan 21, 2021
Merged

delete post functionality on each post#26
Pranav016 merged 1 commit intoALPHAVIO:masterfrom
mak-ux:master

Conversation

@mak-ux
Copy link
Copy Markdown

@mak-ux mak-ux commented Jan 21, 2021

Solves #22

@Pranav016
Copy link
Copy Markdown
Member

I checked the code, seems good. Can you share a screen recording of the working of the feature you added, put it in the comments itself. Since the website isn't deployed yet and I don't have Mongo on my Gitpod so I can't test the feature.

@mak-ux
Copy link
Copy Markdown
Author

mak-ux commented Jan 21, 2021 via email

@mak-ux
Copy link
Copy Markdown
Author

mak-ux commented Jan 21, 2021

blog_post_delete.mp4

@Pranav016
Copy link
Copy Markdown
Member

blog_post_delete.mp4

Excellent work @mak-ux . I am merging this time but just keep in mind next time that work on a new branch before making a PR. Never commit to master and write good commit messages. These are the best practices. Happy Learning!

@Pranav016 Pranav016 merged commit f99d60a into ALPHAVIO:master Jan 21, 2021
@Pranav016
Copy link
Copy Markdown
Member

@all-contributors please add @mak-ux for code

@allcontributors
Copy link
Copy Markdown

@Pranav016

I've put up a pull request to add @mak-ux! 🎉

@Pranav016
Copy link
Copy Markdown
Member

@all-contributors please add @mak-ux for code

@allcontributors
Copy link
Copy Markdown

@Pranav016

I've put up a pull request to add @mak-ux! 🎉

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