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

Fix soft delete check, null check #17

Merged
merged 2 commits into from
Dec 3, 2021
Merged

Fix soft delete check, null check #17

merged 2 commits into from
Dec 3, 2021

Conversation

hi019
Copy link
Collaborator

@hi019 hi019 commented Dec 3, 2021

Change Summary

@hi019 hi019 marked this pull request as ready for review December 3, 2021 16:10
@manavo
Copy link
Contributor

manavo commented Dec 3, 2021

Looks good to me

@hi019 hi019 merged commit cf1c1de into master Dec 3, 2021
@hi019 hi019 deleted the 15-review-fixes branch December 3, 2021 18:50
@manavo
Copy link
Contributor

manavo commented Dec 6, 2021

@hi019 can we tag a new release with these changes? Thanks!

@hi019
Copy link
Collaborator Author

hi019 commented Dec 6, 2021

Yep, will do it later today

@manavo
Copy link
Contributor

manavo commented Dec 6, 2021

Thanks!

@hi019
Copy link
Collaborator Author

hi019 commented Dec 6, 2021

Released

@manavo
Copy link
Contributor

manavo commented Dec 6, 2021

Thanks!

@hi019
Copy link
Collaborator Author

hi019 commented Dec 6, 2021

By the way, would you be interested in becoming a maintainer of this repository @manavo ?

@manavo
Copy link
Contributor

manavo commented Dec 6, 2021

Thanks for the offer @hi019, really appreciate it! Unfortunately it's not something I can commit to at the moment, but if I have more time in the future would love to discuss again!

@hi019
Copy link
Collaborator Author

hi019 commented Dec 7, 2021

Sounds good!

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