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: adds nofollow from config #675

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

eduardoformiga
Copy link
Member

@eduardoformiga eduardoformiga commented Jan 30, 2025

What's the purpose of this pull request?

How does it work?

How to test it?

Faststore related PRs

References

@eduardoformiga eduardoformiga requested a review from a team as a code owner January 30, 2025 15:00
Copy link

vercel bot commented Jan 30, 2025

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

Name Status Preview Updated (UTC)
starter ✅ Ready (Inspect) Visit Preview Jan 30, 2025 3:07pm

eduardoformiga added a commit to vtex/faststore that referenced this pull request Jan 31, 2025
## What's the purpose of this pull request?

This PR aims to enable the `nofollow` prop from NextSeo from the
discovery.config value.

## How it works?

The default value is true as was previously, but the stores can switch
to false as needed.

## How to test it?

You can switch the value from `discovery.config.js `->
`seo.search.noFollow` and check the value from the meta tag related.

### Starters Deploy Preview

- vtex-sites/starter.store#675

| [nofollow
true](https://storeframework-cm652ufll028lmgv665a6xv0g-dgphn5fzx.b.vtex.app/s?q=headphone&fuzzy=0&operator=and&facets=fuzzy%2Coperator&sort=score_desc&page=0)
| [nofollow
false](https://storeframework-cm652ufll028lmgv665a6xv0g-7mt68ho4g.b.vtex.app/s?q=headphone&fuzzy=0&operator=and&facets=fuzzy%2Coperator&sort=score_desc&page=0)
|
|--------|--------|
| <img width="1274" alt="Screenshot 2025-01-30 at 12 08 55"
src="https://github.com/user-attachments/assets/0e2a542c-cc97-47c5-a4e1-b8110d47076f"
/> | <img width="1297" alt="Screenshot 2025-01-30 at 12 05 31"
src="https://github.com/user-attachments/assets/498411b9-86ef-4da4-9e8d-47dfe89f653d"
/> |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant