Multi-cache

[Gallophostrix]

Shoud work with the refactor (similar to #40)

I added new arguments:

• cache-list, which allows you to see all the cached files,
• delete, which allows you to delete one cached element,
• clear, which deletes all the cache

[Gallophostrix]

I don't know if it's up to date with other PRs tho!

Btw, the cached files are saved with a relative or an absolute path, but both work after caching (as the file is saved in assets). Some bugs might maybe happen if you delete the original file, not sure yet

I'd like to check it first with you, but the new path system might solve #36

[Gallophostrix]

@supplefrog could you maybe try this version? You should directly be able to use example.mp4

[supplefrog]

@Gallophostrix all good now. The functionality reflected in the readme would be nice.

[Gallophostrix]

@Gallophostrix all good now. The functionality reflected in the readme would be nice.

Great, I'm glad it helped!

For the readme file, do you mean the multi-cache functionality doesn't appear? If that's the case, I'm planning to add some further information about it 👍.
Or do you mean that something in the readme doesn't exist in the code?

Anyway, thanks for the feedback!

[supplefrog]

For the readme file, do you mean the multi-cache functionality doesn't appear? If that's the case, I'm planning to add some further information about it 👍. Or do you mean that something in the readme doesn't exist in the code?

Sorry for being ambiguous. I meant the functionality to use example.mp4 directly, to be documented in the readme.

[Gallophostrix]

Sorry for being ambiguous. I meant the functionality to use example.mp4 directly, to be documented in the readme.

Yes, you're right, that should be added, thank you for the advice!

[Notenlish]

Hey is this PR ready to be merged, are there any missing functionality etc.?

[Gallophostrix]

Hey is this PR ready to be merged, are there any missing functionality etc.?

Hey! Everything should be alright afaik 👍

[Notenlish]

Overall I like the PR, this multi-cache thing works great. I will merge it once you do the fixes for the points I mentioned in my comments.

[Gallophostrix]

Sorry, I've been pretty busy the last few weeks, I'll take into account your comments and update the code accordingly 👍

[Gallophostrix]

Hey @Notenlish ! Could you check this update ? I added some requested changes, with some comments to your requests as well!

[Notenlish]

Everything seems to be correct, but I can't merge it. There is a merge conflict in the bash script.

[Gallophostrix]

Finally should be alright!

[Notenlish]

Thanks for your contribution!