Skip to content

Conversation

@anshulagrawal2902
Copy link

Purpose of PR?:

Fixes #2506

Does this PR introduce a breaking change?
No

If the changes in this PR are manually verified, list down the scenarios covered::

Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Does this PR results in some Documentation changes?
If yes, include the list of Documentation changes

Checklist:

  • Bug fix. Fixes #
  • New feature (Non-API breaking changes that adds functionality)
  • PR Title follows the convention of <type>: <subject>
  • Commit has unit tests

@ReimarBauer ReimarBauer requested a review from matrss March 15, 2025 06:14

def create_app(name="", imprint=None, gdpr=None):
imprint_file = imprint
gdpr_file = gdpr
Copy link
Member

@ReimarBauer ReimarBauer Mar 15, 2025

Choose a reason for hiding this comment

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

see flake8 test

@ReimarBauer
Copy link
Member

ReimarBauer commented Mar 15, 2025

Welcome @anshulagrawal2902

Thx for working on this. Have a look on the failing tests.

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.

Use the flask-native global configuration store instead of rolling our own

2 participants