IBX-9147: Symbol attribute type described in Developer Documentation - v4.6#2525
Conversation
mnocon
left a comment
There was a problem hiding this comment.
Thank you for preparing this!
I've left a couple of suggestions to make it clearer - but I believe it already includes all the neccessary parts, we just need to improve them a bit.
I cannot provide you with the PostgreSQL schema, we will need someone from the backend team for that.
I will take care of it. |
9685412 to
2bc62c6
Compare
Preview of modified filesPreview of modified Markdown: |
mnocon
left a comment
There was a problem hiding this comment.
Looks good overall, just a couple of comments to improve this.
Please remember to create a PR to the user doc, mentioning this attribute (just a mention is ok, we did the same for date and time: https://doc.ibexa.co/projects/userguide/en/latest/pim/work_with_product_attributes/ )
Also, please have a look at the read the docs error - I think we (I) have lost the mkdocs.yml modification during conflict resolution, could you please readd the symbolattribute_criterion.md file to make the build pass?
|
Another thing which I forgot about: please add the new LTS update to the table of LTS updates (https://github.com/ibexa/documentation-developer/blob/4.6/docs/ibexa_products/editions.md#lts-updates) |
…- v4.6 (#2525) * Symbol attribute type described * Fixes after review * PHP CS Fixes * Resolved conflicts in product search criteria * Added composer dependency * fixes * review fixes * PHP & JS CS Fixes * symbol_attribute_type.md moved to Attributes folder * fixes; mysql added * fix in the table * fix --------- Co-authored-by: julitafalcondusza <julitafalcondusza@users.noreply.github.com> Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com>
* Added new Twig Component group (#2790) * Twig Component follow ups (#2792) * Added Priority for YAML configuration * Added doc for storefront groups * Adjusted paths * Update docs/templating/layout/customize_storefront_layout.md Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> --------- Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> * [Twig Components] Added Menu component (#2799) * Docs: Add Menu component to Twig Components documentation * Docs: Extend Twig Components YAML example with Menu component * Discounts 4.6.22 (#2810) * Adjusted code samples * [Discounts] Added doc for 4.6.22 * Fixed limit mention * Added update mention * Removed mentions of OverridePrioritatization * Wording * Update code_samples/discounts/src/Command/ManageDiscountsCommand.php Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * Fixed highlights * Review suggestions * Added missing v prefix --------- Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * IBX-9147: Symbol attribute type described in Developer Documentation - v4.6 (#2525) * Symbol attribute type described * Fixes after review * PHP CS Fixes * Resolved conflicts in product search criteria * Added composer dependency * fixes * review fixes * PHP & JS CS Fixes * symbol_attribute_type.md moved to Attributes folder * fixes; mysql added * fix in the table * fix --------- Co-authored-by: julitafalcondusza <julitafalcondusza@users.noreply.github.com> Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com> * Update 4.6.22 (#2851) * Adjusted code samples * [Discounts] Added doc for 4.6.22 * Fixed limit mention * Added update mention * Removed mentions of OverridePrioritatization * Wording * Update code_samples/discounts/src/Command/ManageDiscountsCommand.php Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * Fixed highlights * Review suggestions * Added missing v prefix * Added doc for IBX-9933 * Fixed badge --------- Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * IBX-9845: Solr9 (#2866) * Adjusted code samples * [Discounts] Added doc for 4.6.22 * Fixed limit mention * Added update mention * Removed mentions of OverridePrioritatization * Wording * Update code_samples/discounts/src/Command/ManageDiscountsCommand.php Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * Fixed highlights * Review suggestions * Added missing v prefix * Described requirements * Added Solr 9 instructions * Fixed link * Updated instructions * Apply suggestions from code review Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> --------- Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> * Release notes (#2869) * IBX-10262: Added mention of the renamed class * Release notes for 4.6.22 * Apply suggestions from code review Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> * Review fixes * Fixed link * Last minute fixes --------- Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> * Regenerated baseline * Removed bullet * Bumped latest tag * Added PHP API links * Added mention about Symbol attribute * Fixed link * Moved PHP API Symbol links * symbol_attribute_type.md: Link to PHP API Ref --------- Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> Co-authored-by: julitafalcondusza <julitafalcondusza@users.noreply.github.com> Co-authored-by: Adrien Dupuis <61695653+adriendupuis@users.noreply.github.com>
* Added new Twig Component group (#2790) * Twig Component follow ups (#2792) * Added Priority for YAML configuration * Added doc for storefront groups * Adjusted paths * Update docs/templating/layout/customize_storefront_layout.md Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> --------- Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> * [Twig Components] Added Menu component (#2799) * Docs: Add Menu component to Twig Components documentation * Docs: Extend Twig Components YAML example with Menu component * Discounts 4.6.22 (#2810) * Adjusted code samples * [Discounts] Added doc for 4.6.22 * Fixed limit mention * Added update mention * Removed mentions of OverridePrioritatization * Wording * Update code_samples/discounts/src/Command/ManageDiscountsCommand.php Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * Fixed highlights * Review suggestions * Added missing v prefix --------- Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * IBX-9147: Symbol attribute type described in Developer Documentation - v4.6 (#2525) * Symbol attribute type described * Fixes after review * PHP CS Fixes * Resolved conflicts in product search criteria * Added composer dependency * fixes * review fixes * PHP & JS CS Fixes * symbol_attribute_type.md moved to Attributes folder * fixes; mysql added * fix in the table * fix --------- Co-authored-by: julitafalcondusza <julitafalcondusza@users.noreply.github.com> Co-authored-by: Marek Nocoń <mnocon@users.noreply.github.com> * Update 4.6.22 (#2851) * Adjusted code samples * [Discounts] Added doc for 4.6.22 * Fixed limit mention * Added update mention * Removed mentions of OverridePrioritatization * Wording * Update code_samples/discounts/src/Command/ManageDiscountsCommand.php Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * Fixed highlights * Review suggestions * Added missing v prefix * Added doc for IBX-9933 * Fixed badge --------- Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * IBX-9845: Solr9 (#2866) * Adjusted code samples * [Discounts] Added doc for 4.6.22 * Fixed limit mention * Added update mention * Removed mentions of OverridePrioritatization * Wording * Update code_samples/discounts/src/Command/ManageDiscountsCommand.php Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> * Fixed highlights * Review suggestions * Added missing v prefix * Described requirements * Added Solr 9 instructions * Fixed link * Updated instructions * Apply suggestions from code review Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> --------- Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> * Release notes (#2869) * IBX-10262: Added mention of the renamed class * Release notes for 4.6.22 * Apply suggestions from code review Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> * Review fixes * Fixed link * Last minute fixes --------- Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> * Regenerated baseline * Removed bullet * Bumped latest tag * Added PHP API links * Added mention about Symbol attribute * Fixed link * Moved PHP API Symbol links * symbol_attribute_type.md: Link to PHP API Ref --------- Co-authored-by: julitafalcondusza <117284672+julitafalcondusza@users.noreply.github.com> Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co> Co-authored-by: julitafalcondusza <julitafalcondusza@users.noreply.github.com> Co-authored-by: Adrien Dupuis <61695653+adriendupuis@users.noreply.github.com>
Symbol attribute type described in Developer Documentation - v4.6
Preview:
Checklist