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

Build: rename PDF/ePUB filename to valid one automatically #11198

Merged
merged 15 commits into from
Jul 11, 2024

Conversation

humitos
Copy link
Member

@humitos humitos commented Mar 8, 2024

Currently, we require the user to save the PDF or ePUB file with a pretty specific filename: {project.slug}.{extension}. This has caused a lot of confusions to users.

Since we only support 1 file for PDF/ePUB for now, we can rename this file to the valid filename automatically.

Closes #10873

Currently, we _require_ the user to save the PDF or ePUB file with a pretty
specific filename: `{project.slug}-{version.slug}.{extension}`. This has caused
a lot of confusions to users.

Since we only support 1 file for PDF/ePUB for now, we can rename this file to
the valid filename automatically.

Closes #10873
@humitos humitos marked this pull request as ready for review June 17, 2024 11:05
@humitos humitos requested a review from a team as a code owner June 17, 2024 11:05
@humitos humitos requested a review from stsewd June 17, 2024 11:05
readthedocs/projects/tasks/builds.py Outdated Show resolved Hide resolved
readthedocs/projects/tasks/builds.py Outdated Show resolved Hide resolved
@humitos humitos requested a review from stsewd June 27, 2024 08:31
@humitos
Copy link
Member Author

humitos commented Jul 8, 2024

This is ready to re-review and merge 👍🏼

@humitos humitos requested a review from stsewd July 9, 2024 11:38
@humitos humitos requested a review from stsewd July 9, 2024 14:34
@humitos humitos enabled auto-merge (squash) July 9, 2024 14:36
@humitos humitos merged commit 7d40ba9 into main Jul 11, 2024
5 checks passed
@humitos humitos deleted the humitos/allow-any-filename-pdf-epub branch July 11, 2024 09:44
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.

Build: don't care about the filename for the offline formats
2 participants