WEBBDC-2835: Add read-only phones/mails/urls datagrids to the contact section - #91
Open
boulch wants to merge 2 commits into
Open
WEBBDC-2835: Add read-only phones/mails/urls datagrids to the contact section#91boulch wants to merge 2 commits into
boulch wants to merge 2 commits into
Conversation
📝 WalkthroughWalkthroughThe contact section now uses shared contact-grid contracts and helpers. Forms load related contact information into read-only grids. Visible columns control stored rows and rendered phones, mails, URLs, and VAT content. ChangesContact information grids
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant Editor
participant ContactCustomEditForm
participant ContactInformationsGridMixin
participant Directory
participant ContactSection
Editor->>ContactCustomEditForm: click load_contact_informations
ContactCustomEditForm->>ContactInformationsGridMixin: process related_contacts
ContactInformationsGridMixin->>Directory: request contact information
Directory-->>ContactInformationsGridMixin: return phones, mails, and URLs
ContactInformationsGridMixin->>ContactSection: update rows and visible_columns
ContactSection-->>Editor: render selected contact information
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
src/imio/smartweb/core/contents/sections/contact/macros.pt (1)
110-155: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winDuplicated social-icon chain.
Lines 110-129 and 136-155 are the same six-branch type→icon mapping; adding or renaming a network now means editing both copies, and they will drift silently (the exhaustive tests only cover
metal:define-macroandmetal:use-macroit from both the linked and icon-only paths — or, better, drive it from a{type: icon-class}mapping exposed byContactPropertiesand render a single<i tal:attributes="class ...">.♻️ Sketch: one macro, two call sites
<metal:block define-macro="social_icon"> <i tal:define="klass python:{'facebook': 'bi-facebook', 'twitter': 'bi-twitter', 'instagram': 'bi-instagram', 'youtube': 'bi-youtube', 'pinterest': 'bi-pinterest', 'linkedin': 'bi-linkedin'}.get(url.get('type'))" tal:condition="klass" tal:attributes="class string:bi ${klass}"></i> </metal:block>then
<tal:icon tal:condition="python:'type' in columns"><metal:use use-macro="template/macros/social_icon" /></tal:icon>in both places.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/imio/smartweb/core/contents/sections/contact/macros.pt` around lines 110 - 155, Consolidate the duplicated social type-to-icon mapping used by the linked icon block and icon-only block into one reusable `social_icon` macro, or a single mapping supplied by `ContactProperties`. Update both `tal:icon` and `tal:icon_only` call sites to reuse it, preserving the existing six network mappings and rendering no icon for unsupported types.src/imio/smartweb/core/contents/sections/contact/utils.py (1)
52-58: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winTwo sources of truth for the column tuples.
CONTACT_ROW_COLUMNSmirrors the*DisplayColumnsVocabularyFactory.columnstuples insrc/imio/smartweb/core/vocabularies.py(lines 929-954), enforced only by this comment. Drift is silent: a token added to a vocabulary but not here gets filtered out byset(columns) & all_columnsindisplayed_rows, and the existing vocabulary tests assert literal token lists rather than agreement between the two. Cheapest guard is a test asserting equality both ways (importingutilsfromvocabulariesrisks an import cycle).♻️ Suggested test guard (e.g. in tests/test_vocabularies.py)
def test_display_columns_match_the_row_schema(self): from imio.smartweb.core.contents.sections.contact.utils import ( CONTACT_ROW_COLUMNS, ) for kind, name in ( ("phones", "PhoneDisplayColumns"), ("mails", "MailDisplayColumns"), ("urls", "UrlDisplayColumns"), ): factory = getUtility( IVocabularyFactory, "imio.smartweb.vocabulary.{}".format(name) ) self.assertEqual( [term.token for term in factory()], list(CONTACT_ROW_COLUMNS[kind]), kind, )🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/imio/smartweb/core/contents/sections/contact/utils.py` around lines 52 - 58, The vocabulary tests need a regression guard ensuring display-column vocabularies stay synchronized with CONTACT_ROW_COLUMNS. Add a test in the existing vocabulary test suite that imports CONTACT_ROW_COLUMNS locally, retrieves the PhoneDisplayColumns, MailDisplayColumns, and UrlDisplayColumns factories, and asserts each factory’s token list equals the corresponding row-schema tuple in both order and contents.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/imio/smartweb/core/contents/sections/contact/utils.py`:
- Around line 113-132: Update get_remote_contacts to percent-encode every
submitted UID when constructing the directory `@search` query, using
urllib.parse.urlencode rather than interpolating values into the URL. Preserve
UID ordering and the existing language filter, and adjust URL-based tests to
expect the encoded query values.
---
Nitpick comments:
In `@src/imio/smartweb/core/contents/sections/contact/macros.pt`:
- Around line 110-155: Consolidate the duplicated social type-to-icon mapping
used by the linked icon block and icon-only block into one reusable
`social_icon` macro, or a single mapping supplied by `ContactProperties`. Update
both `tal:icon` and `tal:icon_only` call sites to reuse it, preserving the
existing six network mappings and rendering no icon for unsupported types.
In `@src/imio/smartweb/core/contents/sections/contact/utils.py`:
- Around line 52-58: The vocabulary tests need a regression guard ensuring
display-column vocabularies stay synchronized with CONTACT_ROW_COLUMNS. Add a
test in the existing vocabulary test suite that imports CONTACT_ROW_COLUMNS
locally, retrieves the PhoneDisplayColumns, MailDisplayColumns, and
UrlDisplayColumns factories, and asserts each factory’s token list equals the
corresponding row-schema tuple in both order and contents.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 9045d81e-3d1d-4d51-9b13-c33fac7cdf44
📒 Files selected for processing (13)
CHANGES.rstsrc/imio/smartweb/core/contents/sections/contact/content.pysrc/imio/smartweb/core/contents/sections/contact/forms.pysrc/imio/smartweb/core/contents/sections/contact/macros.ptsrc/imio/smartweb/core/contents/sections/contact/utils.pysrc/imio/smartweb/core/tests/resources/json_contact_informations_raw_mock.jsonsrc/imio/smartweb/core/tests/test_frozen_label.pysrc/imio/smartweb/core/tests/test_section_contact.pysrc/imio/smartweb/core/tests/test_section_contact_forms.pysrc/imio/smartweb/core/tests/test_vocabularies.pysrc/imio/smartweb/core/vocabularies.pysrc/imio/smartweb/core/vocabularies.zcmlsrc/imio/smartweb/core/widgets/frozen_label.py
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
src/imio/smartweb/core/contents/sections/contact/macros.pt (1)
110-155: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winDuplicated type→icon mapping (6-way
tal:conditionchain repeated twice).The facebook/twitter/instagram/youtube/pinterest/linkedin icon-selection block is fully duplicated between the "wrapped in
<a>" case (110-129) and the "icon_only" case (136-155). Consider extracting a single icon-class lookup (e.g., a small dict/method on the view returning the bootstrap icon class for a giventype) so the template only needs one conditional icon element, reused in both branches.♻️ Illustrative refactor sketch
- <tal:icon tal:condition="python:'type' in columns"> - <tal:condition tal:condition="python:url.get('type') == 'facebook'"> - <i class="bi bi-facebook"></i> - </tal:condition> - <tal:condition tal:condition="python:url.get('type') == 'twitter'"> - <i class="bi bi-twitter"></i> - </tal:condition> - ... (4 more conditions) ... - </tal:icon> + <i tal:condition="python:'type' in columns and contact.get_social_icon_class(url.get('type'))" + tal:attributes="class python:'bi {}'.format(contact.get_social_icon_class(url.get('type')))"></i>Then reuse the same
<i tal:condition=... >snippet for theicon_onlybranch, eliminating the second full copy.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/imio/smartweb/core/contents/sections/contact/macros.pt` around lines 110 - 155, Remove the duplicated six-way icon-selection chain from the template by introducing one reusable type-to-Bootstrap-icon lookup and a shared icon-rendering element. Update both the linked branch and the icon_only branch to use this lookup for facebook, twitter, instagram, youtube, pinterest, and linkedin, preserving the existing conditions and output.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/imio/smartweb/core/contents/sections/contact/macros.pt`:
- Around line 101-109: Update the social link rendered in the repeated
url_entries block to include rel="noopener noreferrer" alongside target="_blank"
on the anchor element, preserving its existing href, title, and condition
behavior.
- Around line 76-99: Update the phone and mail link attributes and contents in
the repeated phone/mail entries to use an empty-string fallback when number or
mail_address is missing, matching the existing label handling. Ensure missing
values never render as “None” or produce tel:None/mailto:None while preserving
normal values.
---
Nitpick comments:
In `@src/imio/smartweb/core/contents/sections/contact/macros.pt`:
- Around line 110-155: Remove the duplicated six-way icon-selection chain from
the template by introducing one reusable type-to-Bootstrap-icon lookup and a
shared icon-rendering element. Update both the linked branch and the icon_only
branch to use this lookup for facebook, twitter, instagram, youtube, pinterest,
and linkedin, preserving the existing conditions and output.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 17743569-9bcc-4763-ab2b-502ed49da301
📒 Files selected for processing (13)
CHANGES.rstsrc/imio/smartweb/core/contents/sections/contact/content.pysrc/imio/smartweb/core/contents/sections/contact/forms.pysrc/imio/smartweb/core/contents/sections/contact/macros.ptsrc/imio/smartweb/core/contents/sections/contact/utils.pysrc/imio/smartweb/core/tests/resources/json_contact_informations_raw_mock.jsonsrc/imio/smartweb/core/tests/test_frozen_label.pysrc/imio/smartweb/core/tests/test_section_contact.pysrc/imio/smartweb/core/tests/test_section_contact_forms.pysrc/imio/smartweb/core/tests/test_vocabularies.pysrc/imio/smartweb/core/vocabularies.pysrc/imio/smartweb/core/vocabularies.zcmlsrc/imio/smartweb/core/widgets/frozen_label.py
🚧 Files skipped from review as they are similar to previous changes (2)
- src/imio/smartweb/core/tests/test_frozen_label.py
- CHANGES.rst
Comment on lines
+76
to
+99
| <tal:phone tal:repeat="entry phone_entries"> | ||
| <li tal:define="phone entry/data; columns entry/columns" | ||
| tal:attributes="class python:phone.get('type') or ''"> | ||
| <span tal:condition="python:'label' in columns" | ||
| tal:replace="python:phone.get('label') or ''"></span> | ||
| <span tal:condition="python:'type' in columns" | ||
| tal:replace="python:contact.translated_type('phones', phone.get('type'))"></span> | ||
| <a tal:condition="python:'number' in columns" | ||
| tal:attributes="href python:'tel:{}'.format(phone.get('number'))" | ||
| tal:content="python:'{}'.format(phone.get('number'))" /> | ||
| </li> | ||
| </tal:phone> | ||
| <tal:mail tal:repeat="entry mail_entries"> | ||
| <li tal:define="mail entry/data; columns entry/columns" | ||
| tal:attributes="class python:mail.get('type') or ''"> | ||
| <span tal:condition="python:'label' in columns" | ||
| tal:replace="python:mail.get('label') or ''"></span> | ||
| <span tal:condition="python:'type' in columns" | ||
| tal:replace="python:contact.translated_type('mails', mail.get('type'))"></span> | ||
| <a tal:condition="python:'mail_address' in columns" | ||
| tal:attributes="href python:'mailto:{}'.format(mail.get('mail_address'))" | ||
| tal:content="python:'{}'.format(mail.get('mail_address'))" /> | ||
| </li> | ||
| </tal:mail> |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Guard number/mail_address against missing values to avoid literal tel:None / mailto:None.
'tel:{}'.format(phone.get('number')) and '{}'.format(phone.get('number')) render the string "None" when the number key is missing/empty for a row even though the number column is selected — producing a broken tel:None link with visible text "None". Same issue for mail_address/mailto:. Note the label span two lines above already guards with or ''; number/mail_address lack the equivalent fallback.
🐛 Proposed fix
- <a tal:condition="python:'number' in columns"
+ <a tal:condition="python:'number' in columns and phone.get('number')"
tal:attributes="href python:'tel:{}'.format(phone.get('number'))"
- tal:content="python:'{}'.format(phone.get('number'))" />
+ tal:content="python:phone.get('number')" />- <a tal:condition="python:'mail_address' in columns"
+ <a tal:condition="python:'mail_address' in columns and mail.get('mail_address')"
tal:attributes="href python:'mailto:{}'.format(mail.get('mail_address'))"
- tal:content="python:'{}'.format(mail.get('mail_address'))" />
+ tal:content="python:mail.get('mail_address')" />📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| <tal:phone tal:repeat="entry phone_entries"> | |
| <li tal:define="phone entry/data; columns entry/columns" | |
| tal:attributes="class python:phone.get('type') or ''"> | |
| <span tal:condition="python:'label' in columns" | |
| tal:replace="python:phone.get('label') or ''"></span> | |
| <span tal:condition="python:'type' in columns" | |
| tal:replace="python:contact.translated_type('phones', phone.get('type'))"></span> | |
| <a tal:condition="python:'number' in columns" | |
| tal:attributes="href python:'tel:{}'.format(phone.get('number'))" | |
| tal:content="python:'{}'.format(phone.get('number'))" /> | |
| </li> | |
| </tal:phone> | |
| <tal:mail tal:repeat="entry mail_entries"> | |
| <li tal:define="mail entry/data; columns entry/columns" | |
| tal:attributes="class python:mail.get('type') or ''"> | |
| <span tal:condition="python:'label' in columns" | |
| tal:replace="python:mail.get('label') or ''"></span> | |
| <span tal:condition="python:'type' in columns" | |
| tal:replace="python:contact.translated_type('mails', mail.get('type'))"></span> | |
| <a tal:condition="python:'mail_address' in columns" | |
| tal:attributes="href python:'mailto:{}'.format(mail.get('mail_address'))" | |
| tal:content="python:'{}'.format(mail.get('mail_address'))" /> | |
| </li> | |
| </tal:mail> | |
| <tal:phone tal:repeat="entry phone_entries"> | |
| <li tal:define="phone entry/data; columns entry/columns" | |
| tal:attributes="class python:phone.get('type') or ''"> | |
| <span tal:condition="python:'label' in columns" | |
| tal:replace="python:phone.get('label') or ''"></span> | |
| <span tal:condition="python:'type' in columns" | |
| tal:replace="python:contact.translated_type('phones', phone.get('type'))"></span> | |
| <a tal:condition="python:'number' in columns and phone.get('number')" | |
| tal:attributes="href python:'tel:{}'.format(phone.get('number'))" | |
| tal:content="python:phone.get('number')" /> | |
| </li> | |
| </tal:phone> | |
| <tal:mail tal:repeat="entry mail_entries"> | |
| <li tal:define="mail entry/data; columns entry/columns" | |
| tal:attributes="class python:mail.get('type') or ''"> | |
| <span tal:condition="python:'label' in columns" | |
| tal:replace="python:mail.get('label') or ''"></span> | |
| <span tal:condition="python:'type' in columns" | |
| tal:replace="python:contact.translated_type('mails', mail.get('type'))"></span> | |
| <a tal:condition="python:'mail_address' in columns and mail.get('mail_address')" | |
| tal:attributes="href python:'mailto:{}'.format(mail.get('mail_address'))" | |
| tal:content="python:mail.get('mail_address')" /> | |
| </li> | |
| </tal:mail> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/imio/smartweb/core/contents/sections/contact/macros.pt` around lines 76 -
99, Update the phone and mail link attributes and contents in the repeated
phone/mail entries to use an empty-string fallback when number or mail_address
is missing, matching the existing label handling. Ensure missing values never
render as “None” or produce tel:None/mailto:None while preserving normal values.
Comment on lines
101
to
+109
| <ul class="contact_informations_social" | ||
| tal:define="urls python:contact.get_urls" | ||
| tal:condition="urls"> | ||
| <li class="urls" | ||
| tal:repeat="url urls" | ||
| tal:attributes="class python:url.get('type') or ''"> | ||
| <a tal:attributes="href python:url.get('url'); | ||
| title python:contact.get_translated_url_type(url.get('type'))" | ||
| target="_blank"> | ||
| tal:condition="url_entries"> | ||
| <tal:url tal:repeat="entry url_entries"> | ||
| <li tal:define="url entry/data; columns entry/columns" | ||
| tal:attributes="class python:url.get('type') or ''"> | ||
| <a tal:condition="python:'url' in columns" | ||
| tal:attributes="href python:url.get('url'); | ||
| title python:contact.get_translated_url_type(url.get('type')) if ('type' in columns and url.get('type')) else nothing" | ||
| target="_blank"> |
There was a problem hiding this comment.
🔒 Security & Privacy | 🟡 Minor | ⚡ Quick win
Add rel="noopener noreferrer" to the target="_blank" social link.
Opening external URLs with target="_blank" and no rel="noopener noreferrer" exposes the page to reverse-tabnabbing (the opened page can access window.opener).
🛡️ Proposed fix
<a tal:condition="python:'url' in columns"
tal:attributes="href python:url.get('url');
title python:contact.get_translated_url_type(url.get('type')) if ('type' in columns and url.get('type')) else nothing"
+ rel="noopener noreferrer"
target="_blank">📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| <ul class="contact_informations_social" | |
| tal:define="urls python:contact.get_urls" | |
| tal:condition="urls"> | |
| <li class="urls" | |
| tal:repeat="url urls" | |
| tal:attributes="class python:url.get('type') or ''"> | |
| <a tal:attributes="href python:url.get('url'); | |
| title python:contact.get_translated_url_type(url.get('type'))" | |
| target="_blank"> | |
| tal:condition="url_entries"> | |
| <tal:url tal:repeat="entry url_entries"> | |
| <li tal:define="url entry/data; columns entry/columns" | |
| tal:attributes="class python:url.get('type') or ''"> | |
| <a tal:condition="python:'url' in columns" | |
| tal:attributes="href python:url.get('url'); | |
| title python:contact.get_translated_url_type(url.get('type')) if ('type' in columns and url.get('type')) else nothing" | |
| target="_blank"> | |
| <ul class="contact_informations_social" | |
| tal:condition="url_entries"> | |
| <tal:url tal:repeat="entry url_entries"> | |
| <li tal:define="url entry/data; columns entry/columns" | |
| tal:attributes="class python:url.get('type') or ''"> | |
| <a tal:condition="python:'url' in columns" | |
| tal:attributes="href python:url.get('url'); | |
| title python:contact.get_translated_url_type(url.get('type')) if ('type' in columns and url.get('type')) else nothing" | |
| rel="noopener noreferrer" | |
| target="_blank"> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/imio/smartweb/core/contents/sections/contact/macros.pt` around lines 101
- 109, Update the social link rendered in the repeated url_entries block to
include rel="noopener noreferrer" alongside target="_blank" on the anchor
element, preserving its existing href, title, and condition behavior.
…pers, grid form mixin, frozen-label widget and vocabularies to imio.smartweb.common
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@base.cfg`:
- Around line 70-71: Update the imio.smartweb.common source entry under
[sources] in base.cfg to use the protected main branch, or remove the override
so the inherited main configuration applies. Keep WEBBDC-2835 branch overrides
only in stack-specific configuration.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: e78a2f09-46b5-49b6-a5b2-3c7a3ae23cfd
📒 Files selected for processing (5)
CHANGES.rstbase.cfgsrc/imio/smartweb/core/contents/sections/contact/content.pysrc/imio/smartweb/core/contents/sections/contact/forms.pysrc/imio/smartweb/core/contents/sections/contact/utils.py
🚧 Files skipped from review as they are similar to previous changes (4)
- src/imio/smartweb/core/contents/sections/contact/content.py
- src/imio/smartweb/core/contents/sections/contact/utils.py
- CHANGES.rst
- src/imio/smartweb/core/contents/sections/contact/forms.py
Comment on lines
+70
to
+71
| [sources] | ||
| imio.smartweb.common = git https://github.com/IMIO/imio.smartweb.common.git pushurl=git@github.com:IMIO/imio.smartweb.common.git branch=WEBBDC-2835 No newline at end of file |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
printf '%s\n' '--- tracked config files ---'
git ls-files '*base.cfg' '*sources.cfg' '*buildout*.cfg' | sed -n '1,120p'
printf '%s\n' '--- base.cfg relevant section ---'
base=$(git ls-files | awk '$0=="base.cfg"{print; exit}')
if [ -n "$base" ]; then
cat -n "$base" | sed -n '55,85p'
else
echo 'base.cfg not found at repository root'
fi
printf '%s\n' '--- source configuration references ---'
rg -n -C 3 'imio\.smartweb\.common|sources\.cfg|extends|WEBBDC-2835' --glob '*.cfg' --glob '*.ini' --glob '*.txt' --glob '*.md' . | sed -n '1,240p'Repository: IMIO/imio.smartweb.core
Length of output: 4798
🏁 Script executed:
printf '%s\n' '--- repository state ---'
git status --short --branch
printf '%s\n' '--- inherited source configuration on main ---'
curl -fsSL https://raw.githubusercontent.com/IMIO/buildout.smartweb/main/sources.cfg |
grep -n -C 2 'imio\.smartweb\.common' || true
printf '%s\n' '--- dependency branch metadata ---'
curl -fsSL 'https://api.github.com/repos/IMIO/imio.smartweb.common/branches/WEBBDC-2835' |
jq '{name, sha: .commit.sha, protected}' || true
printf '%s\n' '--- current dependency main metadata ---'
curl -fsSL 'https://api.github.com/repos/IMIO/imio.smartweb.common/branches/main' |
jq '{name, sha: .commit.sha, protected}' || trueRepository: IMIO/imio.smartweb.core
Length of output: 1193
Keep the default buildout on main.
base.cfg overrides the inherited imio.smartweb.common source with the unprotected WEBBDC-2835 branch. After the common-package changes merge, remove this override or set branch=main. Keep ticket-branch overrides in stack-specific configuration.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@base.cfg` around lines 70 - 71, Update the imio.smartweb.common source entry
under [sources] in base.cfg to use the protected main branch, or remove the
override so the inherited main configuration applies. Keep WEBBDC-2835 branch
overrides only in stack-specific configuration.
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
1 participant
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.
Summary by CodeRabbit
New Features
Bug Fixes
Documentation