-
Notifications
You must be signed in to change notification settings - Fork 5
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
UHF-7298 Remove infinite scroll #433
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ook that removes the configuration from database
…ode and the configuration
…o be deleted exists
This was referenced Feb 7, 2023
…f updateExistingConfig to override values that weren't overriden before
…ort of active configuration.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
khalima
approved these changes
Feb 9, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works nicely. 🚀
This was referenced Feb 9, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
UHF-7298
What was done
How to install
git pull origin dev
make fresh
make drush-cim && make drush-uli
composer require drupal/helfi_platform_config:dev-UHF-7298_remove_infinite_scroll && composer require drupal/helfi_tpr:dev-UHF-7298_remove_infinite_scroll && composer require drupal/hdbt:dev-UHF-7298_remove_infinite_scroll
make drush-cr
make drush-updb drush-cr
How to test
composer create-project City-of-Helsinki/drupal-helfi-platform:dev-UHF-7219 test --no-interaction --repository https://repository.drupal.hel.ninja/
andmake new
and checking that the new installation is run successfully and that you can enable the helfi_tpr without problems and that the new views will have full pager. This could be also reviewed from the code but testing it is always better :)Other PRs