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

Feat: integrate with client-cms #172

Merged
merged 50 commits into from
Aug 19, 2022

Conversation

ArthurTriis1
Copy link
Contributor

@ArthurTriis1 ArthurTriis1 commented Aug 11, 2022

Signed-off-by: Arthur Andrade arthurfelandrade@gmail.com## What's the purpose of this pull request?

*this PR is a refactoring of feat: integrate with VTEX headless CMS #104 by @danzanzini using the client-cms.

How does it work?

This PR integrates the Faststore Starter with VTEX headless CMS using the Client CMS Package.
The integration was done in index page, where with the gatsby getServerData function all data was fetched, and the page was mounted.

Was added a src/cms/cmsHomeFallback.ts file as fallback for cases of fail in a request or for a project that don't use CMS integration.

The integration with GlobalCMS items like the Alert should be done in other task.

How to test it?

Open the preview link, visit the index page (/) and see if it's everything fine.

References

Client CMS Package
getServerData function

Checklist

You may erase this after checking them all ;)

PR Title and Commit Messages

PR Description

  • Added a label according to the PR goal - Breaking change, Enhancement, Bug or Chore.
  • Added the component, hook, or pathname in-between backticks (``) - If applicable. E.g., ComponentName component.
  • Identified the function or parameter in the PR - If applicable. E.g., useWindowDimensions hook.

Documentation

  • PR description
  • Added to/Updated the Storybook - if applicable.
  • For documentation changes, ping @ carolinamenezes, @ PedroAntunesCosta or @ Mariana-Caetano to review and update.

@vercel
Copy link

vercel bot commented Aug 11, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
gatsby-store-storybook ✅ Ready (Inspect) Visit Preview Aug 19, 2022 at 4:56PM (UTC)

Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
@vtex-sites
Copy link

vtex-sites bot commented Aug 11, 2022

Preview is ready

This pull request generated a Preview

👀   Preview: https://preview-172--gatsby.preview.vtex.app
🔬   Go deeper by inspecting the Build Logs
📝   based on commit 8532c62
⚠️   User login might not work on this preview.

@vtex-sites
Copy link

vtex-sites bot commented Aug 11, 2022

Lighthouse Reports

Here are the Lighthouse reports of this Pull Request

📝 Based on commit 8532c62

Lighthouse Report by page
📎   /
📎   /apple-magic-mouse/p
📎   /office

Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
@ArthurTriis1 ArthurTriis1 force-pushed the feat/FSSS-352-integrate-with-client-cms branch from 4e12bac to 24c28cb Compare August 11, 2022 16:11
@ArthurTriis1 ArthurTriis1 changed the title Adds cms-webhook file Feat: integrate with client-cms Aug 11, 2022
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
Signed-off-by: Arthur Andrade <arthurfelandrade@gmail.com>
@ArthurTriis1 ArthurTriis1 merged commit 2870d30 into main Aug 19, 2022
@ArthurTriis1 ArthurTriis1 deleted the feat/FSSS-352-integrate-with-client-cms branch August 19, 2022 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Features New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants