Skip to content
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

Dynamic Multistore #2112

Merged

Conversation

VadymSachenko
Copy link
Contributor

PR Description

This PR updates the Dynamic Multistore feature document.
Jira: https://spryker.atlassian.net/browse/DOC-215
TBD

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md

@VadymSachenko VadymSachenko added the updated PR contains meaningful changes to some document label Aug 22, 2023
@VadymSachenko VadymSachenko requested a review from a team as a code owner August 22, 2023 11:14
@VadymSachenko VadymSachenko added the in progress PR is not ready for review label Aug 22, 2023

![store-switcher](https://spryker.s3.eu-central-1.amazonaws.com/docs/pbc/all/dynamic-multistore/dynamic-multistore.md/store-switcher.png)

![search-suggestions](https://spryker.s3.eu-central-1.amazonaws.com/docs/pbc/all/dynamic-multistore/dynamic-multistore.md/search-suggestions.png)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a description to this screenshot


![search-suggestions](https://spryker.s3.eu-central-1.amazonaws.com/docs/pbc/all/dynamic-multistore/dynamic-multistore.md/search-suggestions.png)

![assigned-product-searchable-in-the-storefront](https://spryker.s3.eu-central-1.amazonaws.com/docs/pbc/all/dynamic-multistore/dynamic-multistore.md/assigned-product-searchable-in-the-storefront.png)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a description to this screanshot


![store-relation](https://spryker.s3.eu-central-1.amazonaws.com/docs/pbc/all/dynamic-multistore/dynamic-multistore.md/store-relation.png)

![product-prices](https://spryker.s3.eu-central-1.amazonaws.com/docs/pbc/all/dynamic-multistore/dynamic-multistore.md/product-prices.png)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a description to this screanshot

@VadymSachenko VadymSachenko added PO review needed PR needs PO review and removed in progress PR is not ready for review labels Aug 23, 2023
@VadymSachenko VadymSachenko added ready for merge The PR can be merged and removed PO review needed PR needs PO review labels Aug 28, 2023

Dynamic Multistore is part of an *Early Access Release*. This *Early Access* feature introduces the ability to handle the store entity in the Back Office. Business users can try creating stores without editing the `Stores.php` file and redeploying the system.
In the following sections, you can learn more details about the capabilities of Dynamic Multistore.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What value does this sentence give?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It just seemed right to add an intro sentence for the sections

Co-authored-by: Andrii Tserkovnyi <andrii.tserkovnyi@spryker.com>
CODEOWNERS Outdated Show resolved Hide resolved

Dynamic Multistore is part of an *Early Access Release*. This *Early Access* feature introduces the ability to handle the store entity in the Back Office. Business users can try creating stores without editing the `Stores.php` file and redeploying the system.
In the following sections, you can learn more details about the capabilities of Dynamic Multistore.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It just seemed right to add an intro sentence for the sections

Vadym Sachenko and others added 2 commits August 28, 2023 11:16
Co-authored-by: Andrii Tserkovnyi <andrii.tserkovnyi@spryker.com>
@andriitserkovnyi andriitserkovnyi self-assigned this Aug 30, 2023
@andriitserkovnyi andriitserkovnyi added new PR contains a new document and removed ready for merge The PR can be merged updated PR contains meaningful changes to some document labels Aug 30, 2023

The number of tables that use relation with the `spy_store` table may depend on the use of the functionality, and on the installed modules.

The list of available tables with data that may contain a foreign key relationship:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What should I do with it?

@@ -86,94 +95,94 @@ The list of available tables with data that may contain a foreign key relationsh
- `spy_touch_search`
- `spy_touch_storage`

Store has new configuration tables that were used for dynamic store:
Store has new configuration tables that are used for dynamic store:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What should I do with these tables?

The following indexes are available in the standard configuration (example: `xxx` - store name):
Index name components are delimited with an underscore—for example, `spryker_xxx_page`.

The following indexes are available in the standard configuration:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What should I do with them?

```json
{"stores":["AT","DE","XXX"],"_timestamp":111111111111}
```
- Delete key `kv:store:xxx` with value about store.
3. Delete the `kv:store:xxx` key with the store value.


Note: A project's custom data stored keys may also be present.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What are we trying to explain here?

- `kv:price_product_abstract_merchant_relationship:xxx:*`


2. Adjust `kv:store_list:` and delete the store name XXX values in stores.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What should I delete? Can you provide examples of code before and after it's deleted?

```json
{"stores":["AT","DE","XXX"],"_timestamp":111111111111}
```
- Delete key `kv:store:xxx` with value about store.
3. Delete the `kv:store:xxx` key with the store value.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What should I delete? Can you provide examples of code before and after it's deleted?

@andriitserkovnyi andriitserkovnyi merged commit 3b34485 into master Sep 20, 2023
@andriitserkovnyi andriitserkovnyi deleted the DOC-215-review-and-publish-dynamic-multistore-doc branch September 20, 2023 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new PR contains a new document
Development

Successfully merging this pull request may close these issues.

3 participants