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

[~] BO : Remove open graph title/description #4347

Merged
merged 1 commit into from Oct 28, 2015
Merged

[~] BO : Remove open graph title/description #4347

merged 1 commit into from Oct 28, 2015

Conversation

kelu95
Copy link
Contributor

@kelu95 kelu95 commented Oct 28, 2015

Also, this features already exists on seo expert module

xGouley added a commit that referenced this pull request Oct 28, 2015
[~] BO : Remove open graph title/description
@xGouley xGouley merged commit 85d1681 into PrestaShop:develop Oct 28, 2015
@inetbiz
Copy link
Contributor

inetbiz commented Oct 28, 2015

I am against removing OGP tags from core and native social share module. Or is this a revert of the social sharing commit that I was so strongly against?

@vincentbz
Copy link
Contributor

Hi,

We removed it because it has no sense to leave it in the core while it's not something essential for most of the users and there are many modules doing it.

@inetbiz
Copy link
Contributor

inetbiz commented Oct 28, 2015

@vincentbz When I review the pull, it's just taking out those custom BO features to change the title and description. I don't see any OGP tag work in the pull. Did I miss something?

@vincentbz
Copy link
Contributor

We don't modify at all the native module. We planned to add something that would be a duplicate of SEO expert module or other module. And we removed it

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.

None yet

4 participants