- Issue created by @YevKo
The code works as intended, so I don't think it's a bug report. I'll switch the issue to a feature request and will look into implementing it.
- Merge request !8Issue #3366236: Code works only for the view mode "full" → (Merged) created by admirlju
- Status changed to Needs review
almost 2 years ago 1:07pm 11 August 2023 Added a view mode checkbox selection. You can select multiple view modes, by default no view mode is selected so now if you ever want to temporarily remove some entity type you can use that.
I've run the automated test locally and did a bit of testing by hand and it looks like everything works fine.
Rebased the issue fork. The tests are failing because of an issue being fixed in 📌 Drupal 10 Fix deprecations in tests Needs review .
- Status changed to Needs work
over 1 year ago 7:46am 13 October 2023 - Status changed to Needs review
over 1 year ago 11:25am 13 October 2023 Since it's a new functionality, decided to add an automated test for it. The new test is passing, the old one is still failing for the same reason as mentioned before.
- First commit to issue fork.
- First commit to issue fork.
- 🇸🇮Slovenia deaom
Tested the changes with a clean Drupal 11 install and the latest MR. The code adds the newly created custom view mode display in the configuration settings of a recently read settings page and can be selected. As before, multiple view modes can be selected. As automatic tests are passing, tested manually:
- Created a new view mode custom full
- In recently read settings applied the custom full view mode display for article.
- Added 2 content type articles
- Added the recently read block
- Checked there is nothing displayed there when visiting newly created articles in their default view mode
- Programmatically updated the default full view mode to display the custom full for article content type
- Visited the articles and they were displayed in the block.
I'm marking this change as RTBC as the changes added do not affect the functionality, but fell free to change it to needs review if needed.
-
jernejmramor →
committed acf3156a on 8.x-1.x authored by
admirlju →
Issue #3366236 by admirlju, deaom, jernejmramor: Code works only for the...
-
jernejmramor →
committed acf3156a on 8.x-1.x authored by
admirlju →
- 🇸🇮Slovenia jernejmramor
Since this was RTBC and I think it would be a nice feature to have in this module I'm merging this into development branch and this will get included into the next release. Marking this issue as fixed for now, will close it after it gets released.