Skip to content
This repository has been archived by the owner on Dec 5, 2021. It is now read-only.

Fix nuxt.config.js head.link overridden #79

Merged
merged 1 commit into from
Oct 9, 2020

Conversation

SeverinAlexB
Copy link
Contributor

@SeverinAlexB SeverinAlexB commented Oct 9, 2020

Instead of replacing the existing links, this fix makes sure that the Canonical is appended to the existing links.

Fixes #78.

Fixes the problem that head.link defined in nuxt.config.js are overridden.
@TiagoDanin
Copy link
Owner

@Sebubu I believe we do the same with meta, what do you think?

@SeverinAlexB
Copy link
Contributor Author

@Sebubu I believe we do the same with meta, what do you think?

Naaa, the meta tag works as expected. The createMeta(options, this.options.head.meta, template) function takes the existing meta tags and appeneds the nuxt-seo tags to it.

@SeverinAlexB SeverinAlexB removed their assignment Oct 9, 2020
@TiagoDanin TiagoDanin merged commit f2e66ac into TiagoDanin:master Oct 9, 2020
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.

No favicon
3 participants