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

Delete old user picture when he uploads a new one #20

Conversation

Yoann-TYT
Copy link
Contributor

Hello Yann !

Thanks to you I discovered AdoniJs and I love it :)
My first small contribution to fix issue #1

@yann-yinn yann-yinn merged commit 0000713 into yann-yinn:main Oct 5, 2021
@yann-yinn
Copy link
Owner

this works like a charm, thanks @Yoann-TYT for this contribution !
Glad you discovered Adonis

(PS: if you create another PR, make sure to fork dev branch and create the PR against the dev branch - maybe i should configure the dev branch as the default one...)

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