Skip to content

feat: Control features with build arg#75

Open
AaronBenDaniel wants to merge 5 commits into
feature/#80-dockerfilefrom
feature/#74-unified-feature-branch
Open

feat: Control features with build arg#75
AaronBenDaniel wants to merge 5 commits into
feature/#80-dockerfilefrom
feature/#74-unified-feature-branch

Conversation

@AaronBenDaniel

@AaronBenDaniel AaronBenDaniel commented Jun 24, 2025

Copy link
Copy Markdown
Collaborator

Fixes #74.

Merges PDF and non-PDF branches and controls which features are enabled with a build argument.

By default, PDF downloads, the donator frontend, and unlimited bandwidth are disabled. If, while running docker build, --build-arg feature_flag=true--build-arg pdfs=true is added, the features will be enabled.

@AaronBenDaniel AaronBenDaniel self-assigned this Jun 24, 2025
@AaronBenDaniel AaronBenDaniel added the enhancement New feature or request label Jun 24, 2025
Comment thread src/api/src/main.py Outdated
Comment thread src/api/src/main.py Outdated
Comment thread src/frontend/src/routes/+page.svelte Outdated
Comment thread src/frontend/src/routes/+page.svelte Outdated
@AaronBenDaniel

Copy link
Copy Markdown
Collaborator Author

What about the --build-arg?

@TheOnlyWayUp
TheOnlyWayUp changed the base branch from master to feature/#80-dockerfile October 28, 2025 13:19
@TheOnlyWayUp

Copy link
Copy Markdown
Owner

Rebasing to #82. 82 will be merged to master first, and this branch will be next.

@TheOnlyWayUp
TheOnlyWayUp force-pushed the feature/#74-unified-feature-branch branch 2 times, most recently from 6822fb0 to ec16efe Compare October 28, 2025 13:59
@TheOnlyWayUp

Copy link
Copy Markdown
Owner

Rebased the 3 commits on this branch onto 82.
image

@TheOnlyWayUp
TheOnlyWayUp force-pushed the feature/#74-unified-feature-branch branch from ec16efe to 2956399 Compare October 29, 2025 13:13
@AaronBenDaniel

Copy link
Copy Markdown
Collaborator Author

This branch is broken if #82 is merged. Fixes will need to be made before this can get merged into master.

@AaronBenDaniel

Copy link
Copy Markdown
Collaborator Author

I believe this branch is ready to merge. @TheOnlyWayUp can you review it?

@jmjuniorruiz09-dotcom jmjuniorruiz09-dotcom left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hhhh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unified feature branch

3 participants