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 templates from database, move to filesystem #1141

Merged
merged 4 commits into from
Sep 30, 2024
Merged

Remove templates from database, move to filesystem #1141

merged 4 commits into from
Sep 30, 2024

Conversation

tomchop
Copy link
Collaborator

@tomchop tomchop commented Sep 25, 2024

No description provided.

@tomchop tomchop requested a review from udgover September 25, 2024 08:59
Copy link
Collaborator

@udgover udgover left a comment

Choose a reason for hiding this comment

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

LGTM!

@tomchop
Copy link
Collaborator Author

tomchop commented Sep 29, 2024

Waiting for #1137 to be merged so we can use the FileStorageClient

@tomchop tomchop added security-impact Changes that affect security and users should keep an eye on breaking Changes that might break some implementations blocked Items blocked by other changes or issues labels Sep 29, 2024
@tomchop
Copy link
Collaborator Author

tomchop commented Sep 30, 2024

Actually, merging this now:

#1137 (comment)

@tomchop tomchop merged commit 235f703 into main Sep 30, 2024
3 checks passed
@tomchop tomchop deleted the templates branch September 30, 2024 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Items blocked by other changes or issues breaking Changes that might break some implementations security-impact Changes that affect security and users should keep an eye on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants