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

Remove Redis #766

Merged
merged 13 commits into from
Feb 24, 2025
Merged

Remove Redis #766

merged 13 commits into from
Feb 24, 2025

Conversation

tcely
Copy link
Contributor

@tcely tcely commented Feb 24, 2025

No description provided.

@tcely tcely marked this pull request as ready for review February 24, 2025 10:07
@meeb
Copy link
Owner

meeb commented Feb 24, 2025

This might be more sensible to just comment out the run commands and comment out the apt package for now, at some point the worker faff probably needs to be replaced regardless of your excellent hacking on it.

@tcely
Copy link
Contributor Author

tcely commented Feb 24, 2025

Resurrection from git isn't terribly difficult. You can also create a branch for these old files to remain active on instead of main, if you prefer.

The run commands were disabled already.
However, the presence of the services and files left me confused as to how the tasks were actually running when I first looked at the source.

at some point the worker faff probably needs to be replaced regardless of your excellent hacking on it

Do you have an open issue for this work, which could be pinned on GitHub to attract more help and/or ideas?

I would also like to work on the project dashboard, if you're comfortable giving me access to modify that on your repository.

@meeb
Copy link
Owner

meeb commented Feb 24, 2025

Sure, I've created a blank new project on tubesync and added you with write access to it. Is that sufficient? I've never used GitHub projects myself. Agreed on removing redis, it's not a big task to restore it if needed. Thanks as usual!

@meeb meeb merged commit 71b2b15 into meeb:main Feb 24, 2025
@tcely tcely deleted the patch-5 branch February 24, 2025 11:28
@tcely
Copy link
Contributor Author

tcely commented Feb 24, 2025

Sure, I've created a blank new project on tubesync and added you with write access to it. Is that sufficient?

It appears to be. Thanks!

@meeb Would you link the Status project to the tubesync repository and set it to be viewed publicly?

@meeb
Copy link
Owner

meeb commented Feb 24, 2025

I've set the project to public, it appears already to be linked to the repo.

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