Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

mezotv
Copy link
Contributor

@mezotv mezotv commented Oct 6, 2024

Adds support for Pinterest's Rich Pin Data meta tag, while future-proofing for additional Pinterest tags.

This currently works like so

pinterest: {
  richPin: true
}

And would resolve to:

 <meta property="pinterest-rich-pin" content="true" />

Here is pinterests docs on the topic: https://help.pinterest.com/en/business/article/rich-pins

mezotv added 3 commits October 6, 2024 19:31
Adds the pinterest meta tag allowing you to opt out or in of their rich pin data feature
@ijjk ijjk added the type: next label Oct 6, 2024
@mezotv mezotv marked this pull request as draft October 6, 2024 18:11
@mezotv mezotv marked this pull request as ready for review October 7, 2024 05:12
Copy link

@vignesh1507 vignesh1507 left a comment

Choose a reason for hiding this comment

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

Agree with the changes.

@mezotv mezotv closed this Oct 23, 2024
@mezotv mezotv reopened this Oct 23, 2024
@mezotv
Copy link
Contributor Author

mezotv commented Nov 4, 2024

Hey @ijjk what are the chances of this being looked at?

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Failed Failed Secrets high 3   medium 0   low 3   info 0 View in Orca
🔑 The following Secrets have been detected in your pull request across all commits

⚠️ Please take action to mitigate the risk of the identified secrets by revoking them, and if already in use, updating all dependent systems

NAME FILE LINE NUM COMMIT
high GCP OAuth Client Secret ...pp/config/backend.ts 29 484efae View in code
high GCP OAuth Client Secret ...supertokensConfig.js 36 6852eff View in code
high GCP OAuth Client Secret ...ith-supertokens/.env 6 f6c62d3 View in code
low GCP OAuth Client ID ...pp/config/backend.ts 28 484efae View in code
low GCP OAuth Client ID ...supertokensConfig.js 35 6852eff View in code
low GCP OAuth Client ID ...ith-supertokens/.env 5 f6c62d3 View in code

@ijjk ijjk requested a review from huozhi March 7, 2025 17:10
@ijjk ijjk added the tests label Mar 11, 2025
@mezotv mezotv requested a review from huozhi March 11, 2025 16:08
@huozhi huozhi added the CI approved Approve running CI for fork label Mar 11, 2025
@mezotv
Copy link
Contributor Author

mezotv commented Mar 11, 2025

I tried removing the changes made to the readme but the pre commit hook didn't allow that, so nothing I can do.

@huozhi huozhi removed the CI approved Approve running CI for fork label Mar 11, 2025
@huozhi
Copy link
Member

huozhi commented Mar 11, 2025

I can't push changes to this pr, I opened another one to complete the implementation and fix the tests #76988

@mezotv
Copy link
Contributor Author

mezotv commented Mar 11, 2025

I can't push changes to this pr, I opened another one to complete the implementation and fix the tests #76988

Sounds good!

@huozhi huozhi closed this in 75831c0 Mar 11, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 26, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants