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

New feat : Upload image in torrent upload page & torrent detail #60

Merged
merged 26 commits into from
Oct 29, 2023

Conversation

Klaiment
Copy link
Contributor

@Klaiment Klaiment commented Oct 11, 2023

With this update, here are the points that have been modified/added:

  1. Fixed a Docker bug that prevented the correct mounting of the config.js file with Docker by explicitly indicating that it is a file and not a directory.

  2. Added the image upload functionality requested in this report:
    link to the report

  3. Also visible on the torrent detail page.

  4. Added the French locales.

.gitignore Outdated Show resolved Hide resolved
api/Dockerfile Outdated Show resolved Hide resolved
api/src/controllers/torrent.js Outdated Show resolved Hide resolved
client/Dockerfile Outdated Show resolved Hide resolved
client/Dockerfile Outdated Show resolved Hide resolved
config.example.js Outdated Show resolved Hide resolved
docker-compose.dev.yml Outdated Show resolved Hide resolved
client/pages/torrent/[infoHash].js Outdated Show resolved Hide resolved
client/pages/upload.js Show resolved Hide resolved
client/pages/upload.js Outdated Show resolved Hide resolved
@Klaiment
Copy link
Contributor Author

All done.

@CodeMeZone
Copy link

Any ETA on this push?

@abiteman
Copy link

Also looking at an update on a potential push.

@tdjsnelling
Copy link
Owner

Sorry for the delay. Thanks for your changes, everything looks good now. Good job getting this implemented

@tdjsnelling tdjsnelling merged commit 49443a0 into tdjsnelling:master Oct 29, 2023
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.

4 participants