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

[WIP] Cms integration #1325

Closed
wants to merge 3 commits into from
Closed

Conversation

filrak
Copy link
Collaborator

@filrak filrak commented Jun 20, 2018

CMS integration for Vue Storefront.
I'm starting with abstract boilerplate for generic use which will be combined with extensions for each platform (eg contentful, wordpress etc).

The extensions are developed on separate repos

@filrak filrak requested a review from pkarw June 20, 2018 21:47
@pkarw
Copy link
Collaborator

pkarw commented Jun 21, 2018

I got the idea, for me it's just fine. Maybe we should cache the blocks in the indexedDb to not query the server each time? if there will be external API call for getting the blocks as I assume is the case here - it won't be cached by default (only Elastic calls are cached by default). Please consider it.

Some checks are failing - please fix it before the merge.

@filrak
Copy link
Collaborator Author

filrak commented Jun 21, 2018

Yes, it will be saved to indexedDB as any other resource. It's just not finished yet.
I wanted to know your opinion about the whole concept and make it transparent to others before i finish in case it's a stupid idea :D

@DavidRouyer
Copy link
Contributor

So, no Elasticsearch? How do you plan to handle search?

@filrak
Copy link
Collaborator Author

filrak commented Jul 18, 2018

@DavidRouyer I'll adjust my PR to current magento cms integration which is using es :)

@pkarw pkarw added the vs-hackathon Tasks for the Hackathon label Aug 3, 2018
@filrak filrak closed this Aug 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vs-hackathon Tasks for the Hackathon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants