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

Added command to delete previews generated by the app #52

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ariselseng
Copy link
Owner

Needs tests.

@MiningPickaxe
Copy link

Hey, just wanted to bump this a bit :)
It seems like this patch no longer works in the latest versions of this app.
I recieve the following error:

In MissingNamedParameter.php line 17:

  Named parameter "fileid" does not have a bound value.

Would be nice if this could work again

@ariselseng
Copy link
Owner Author

@MiningPickaxe If you know php I would certainly like some help merging this. Most importantly this needs unit tests so that we can minimize risk maintaining it in the future.

@MiningPickaxe
Copy link

@MiningPickaxe If you know php I would certainly like some help merging this. Most importantly this needs unit tests so that we can minimize risk maintaining it in the future.

Sadly I don't :/ I understand just enough of php to do what the command did by hand.

@ariselseng ariselseng force-pushed the introduce-delete-previews-command branch from fdb09a7 to 10e3779 Compare January 30, 2022 09:33
@ariselseng
Copy link
Owner Author

@MiningPickaxe I fixed this now. Use at your own risk.

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