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

[#1405] Cleanup 'show_xyz' booleans from siteconfiguration #609

Merged

Conversation

vaszig
Copy link
Contributor

@vaszig vaszig commented May 8, 2023

This is based on #597

@vaszig vaszig changed the base branch from develop to feature/1397-django-cms-profile-apphookconfig May 8, 2023 10:40
@vaszig vaszig force-pushed the feature/1405-cleanup-booleans-from-siteconfiguration branch from 901b3b1 to a747a4b Compare May 8, 2023 12:56
@codecov-commenter
Copy link

codecov-commenter commented May 8, 2023

Codecov Report

Merging #609 (e46eebb) into feature/1397-django-cms-profile-apphookconfig (3ccd580) will increase coverage by 0.43%.
The diff coverage is 87.09%.

@@                                Coverage Diff                                @@
##           feature/1397-django-cms-profile-apphookconfig     #609      +/-   ##
=================================================================================
+ Coverage                                          95.97%   96.40%   +0.43%     
=================================================================================
  Files                                                575      573       -2     
  Lines                                              19950    19793     -157     
=================================================================================
- Hits                                               19147    19082      -65     
+ Misses                                               803      711      -92     
Impacted Files Coverage Δ
src/open_inwoner/configurations/admin.py 100.00% <ø> (ø)
src/open_inwoner/configurations/models.py 97.41% <ø> (-0.21%) ⬇️
src/open_inwoner/pdc/views.py 88.30% <ø> (+0.19%) ⬆️
src/open_inwoner/utils/context_processors.py 90.90% <ø> (ø)
src/open_inwoner/cms/banner/forms.py 58.33% <58.33%> (ø)
src/open_inwoner/plans/views.py 96.36% <84.21%> (-0.58%) ⬇️
src/open_inwoner/accounts/views/actions.py 100.00% <100.00%> (ø)
src/open_inwoner/cms/banner/cms_plugins.py 100.00% <100.00%> (ø)
.../cms/banner/migrations/0002_banner_image_height.py 100.00% <100.00%> (ø)
src/open_inwoner/cms/banner/models.py 90.00% <100.00%> (ø)
... and 3 more

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@vaszig vaszig force-pushed the feature/1405-cleanup-booleans-from-siteconfiguration branch 2 times, most recently from 5838024 to 69742fb Compare May 9, 2023 07:42
@alextreme
Copy link
Member

Did a quick check with Vasileios and looks good imho

@vaszig vaszig requested a review from Bartvaderkin May 9, 2023 08:42
src/open_inwoner/plans/views.py Outdated Show resolved Hide resolved
Base automatically changed from feature/1397-django-cms-profile-apphookconfig to develop May 11, 2023 09:13
@vaszig vaszig force-pushed the feature/1405-cleanup-booleans-from-siteconfiguration branch from aa6cb86 to fd9a6fd Compare May 11, 2023 09:18
@Bartvaderkin Bartvaderkin merged commit 325d753 into develop May 11, 2023
@Bartvaderkin Bartvaderkin deleted the feature/1405-cleanup-booleans-from-siteconfiguration branch May 11, 2023 10:15
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.

4 participants