[SD-1776] Added site restriction widget for search listing content type. - #823
Merged
MdNadimHossain merged 7 commits intoSep 14, 2026
Merged
Conversation
…en link for image gallery component. (#825) * [SD-1834] Hero image field spelling correction. * [SD-1837] fix broken link for image gallery component.
There was a problem hiding this comment.
🟡 Changes recommended
The new update hook in tide_site_restriction.install contains a namespace casing issue and a PHP syntax error that will break updates.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds/updates configuration to support Tide “site restriction” field widgets on the tide_search_listing node form display, and corrects landing page field UI text by fixing a hero image caption label typo and updating the media gallery help link.
Changes:
- Added an update hook to switch
field_node_primary_siteandfield_node_sitewidgets totide_site_restriction_field_widgeton the search listing form display. - Updated landing page update/config to correct the hero image caption field label spelling.
- Updated media gallery field description to point to the new image gallery documentation link.
File summaries
| File | Description |
|---|---|
| modules/tide_site_restriction/tide_site_restriction.install | Adds update hook to change search listing site field widgets (contains issues noted in PR comments). |
| modules/tide_landing_page/tide_landing_page.install | Adds update hook to correct hero caption label and update media gallery field description link. |
| modules/tide_landing_page/config/install/field.field.paragraph.media_gallery.field_paragraph_media_gallery.yml | Updates media gallery field description URL. |
| modules/tide_landing_page/config/install/field.field.node.landing_page.field_show_hero_image_caption.yml | Fixes “Full-with” → “Full-width” label typo. |
Review details
- Files reviewed: 4/4 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
vincent-gao
approved these changes
Sep 9, 2026
MdNadimHossain
deleted the
bug/SD-1776-fix-site-permissions-for-search-listing
branch
September 14, 2026 06:34
This file contains hidden or 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
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.
JIRA issue: https://digital-vic.atlassian.net/browse/SD-1776
https://digital-vic.atlassian.net/browse/SD-1834
https://digital-vic.atlassian.net/browse/SD-1837
https://digital-vic.atlassian.net/browse/SD-1624
Changed
Screenshots