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
adding source property in sling model of image component to get dam asset path in headless json #1198
Conversation
…the path in crx of the image.......]
"visible": false, | ||
"altText": "abc", | ||
"value": "/content/image-customized.img.png", | ||
"source": "/content/image-customized.img.png", |
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.
The schema validation test case would also fail because you have added a new property for image field type. However, ideally, we should also review and possibly update the specification for source/value property name. Can you start a conversation on our internal Slack channel to finalize the property name? In my opinion, the source should be a custom property that points to the DAM image, given that this detail is specific to AEM. The specification should mandate only a single property for an image, which is its URL (whether the property name should be value or source, we need to decide the same, but only one property should be enforce by spec)
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.
check comments
…the path in crx of the image.......]
b4e86cf
to
f950658
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #1198 +/- ##
============================================
- Coverage 80.89% 80.88% -0.02%
- Complexity 791 793 +2
============================================
Files 92 92
Lines 2146 2150 +4
Branches 292 293 +1
============================================
+ Hits 1736 1739 +3
Misses 253 253
- Partials 157 158 +1 ☔ View full report in Codecov by Sentry. |
Lighthouse scores (desktop)
|
Lighthouse scores (mobile)
|
Accessibility Violations Found
|
2 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
Lighthouse scores (mobile)
|
Lighthouse scores (desktop)
|
Accessibility Violations Found
|
1 similar comment
Accessibility Violations Found
|
Lighthouse scores (mobile)
|
Lighthouse scores (desktop)
|
Accessibility Violations Found
|
2 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
bundles/af-core/src/main/java/com/adobe/cq/forms/core/components/models/form/StaticImage.java
Outdated
Show resolved
Hide resolved
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.
check comments
Lighthouse scores (desktop)
|
Lighthouse scores (mobile)
|
Accessibility Violations Found
|
2 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
No description provided.