ankumalh / assets-commerce

AEM artifacts to be installed while working with new assets-commerce rule engine
0 stars 2 forks source link

Match metadata titles to Commerce #3

Closed meker12 closed 1 day ago

meker12 commented 1 month ago

@ankumalh A couple suggestions to improve the metadata form in the boilerplate repo to have parity with existing Commerce image management:

Suggestion 1:

(Commerce doc reference Image roles)

Suggestion 2:

image

The role isn't too bad since it says: "Choose usage for image" -- but text might be better to say "Select role for image usage"

ankumalh commented 1 month ago

@meker12 - Pablo took care of 1 a while back with https://github.com/ankumalh/assets-commerce/commit/f693023aacd2e404edb285941996f0b627996161

the image in ReadMe needs to be updated though