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

Add edit-only option #66

Merged
merged 8 commits into from
Jun 25, 2023
Merged

Conversation

RFBomb
Copy link
Contributor

@RFBomb RFBomb commented Jun 20, 2023

Includes an info!() comment due to the issue in #35
#61

README.md Outdated Show resolved Hide resolved
src/things/comment.rs Outdated Show resolved Hide resolved
src/things/comment.rs Outdated Show resolved Hide resolved
src/things/comment.rs Outdated Show resolved Hide resolved
@RFBomb
Copy link
Contributor Author

RFBomb commented Jun 25, 2023

For simplicity sake, I created a function for the config that will check both dry run and edit-only mode. It will return TRUE if deletion should be skipped. This allows easily adding on in future if needed too. That way, all the things get the same messages as needed

@andrewbanchich
Copy link
Owner

For simplicity sake, I created a function for the config that will check both dry run and edit-only mode. It will return TRUE if deletion should be skipped. This allows easily adding on in future if needed too. That way, all the things get the same messages as needed

Perfect!

README.md Outdated Show resolved Hide resolved
@andrewbanchich andrewbanchich added the enhancement New feature or request label Jun 25, 2023
@andrewbanchich andrewbanchich merged commit 4a80f0a into andrewbanchich:master Jun 25, 2023
@andrewbanchich andrewbanchich changed the title Prevent Comment Deletion Add edit-only option Jun 25, 2023
@andrewbanchich
Copy link
Owner

Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants