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

refactor: __build_app function clean-up and make all modules accessible #1240

Merged

Conversation

phorward
Copy link
Member

@phorward phorward commented Aug 18, 2024

This is both a refactor and a fix for the Modulename.html = True default that was removed with viur-core 3.6.

The error AttributeError: 'Index' object has no attribute '<modulename>' is fixed with this improvement, and the workaround of conf.main_app.vi.modulename is not neccessary anymore.

Belongs to #1229, replaces #1232.

…ible

This is both a refactor and a fix for the `Modulename.html = True` default that was removed with viur-core 3.6.

The error `AttributeError: 'Index' object has no attribute '<modulename>'` is fixed with this improvement, and the workaround of `conf.main_app.vi.modulename` is not neccessary anymore.
@phorward phorward added bug(fix) Something isn't working or address a specific issue or vulnerability Priority: High After critical issues are fixed, these should be dealt with before any further issues. refactoring Pull requests that refactor code but do not change its behavior. main labels Aug 18, 2024
@phorward phorward added this to the ViUR-core v3.6 milestone Aug 18, 2024
Copy link
Member

@sveneberth sveneberth left a comment

Choose a reason for hiding this comment

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

Works for me

@phorward phorward merged commit 9493f81 into viur-framework:main Aug 21, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug(fix) Something isn't working or address a specific issue or vulnerability main Priority: High After critical issues are fixed, these should be dealt with before any further issues. refactoring Pull requests that refactor code but do not change its behavior.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants