feat: add meta headAttrs

[ricardogobbosouza]

Types of changes

• Bug fix (a non-breaking change which fixes an issue)
• New feature (a non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

Add meta headAttrs

Checklist:

• All new and existing tests are passing.
• My change requires a change to the documentation.
• I have updated the documentation accordingly. (PR: #)
• I have added tests to cover my changes (if not applicable, please state why)

[manniL]

Hey @ricardogobbosouza 👋

As that options isn't documented in vue-meta we should documented it there first before providing an interface for it in Nuxt IMO.

[codecov-io]

Codecov Report

Merging #4536 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #4536      +/-   ##
==========================================
+ Coverage   91.07%   91.07%   +<.01%     
==========================================
  Files          61       61              
  Lines        2184     2186       +2     
  Branches      527      527              
==========================================
+ Hits         1989     1991       +2     
  Misses        177      177              
  Partials       18       18

Impacted Files	Coverage Δ
packages/vue-renderer/src/renderer.js	95.62% <100%> (+0.02%)	⬆️
packages/vue-renderer/src/spa-meta.js	85.45% <100%> (+0.26%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 657a6cc...c2fe1dc. Read the comment docs.

[ricardogobbosouza]

Hi @manniL, i agree
There is this option but it is not documented

[ricardogobbosouza]

I created a PR nuxt/vue-meta#294

[manniL]

vue-meta PR merged.

[pi0]

/ping @Atinux about release

[manniL]

@pi0 There is no vue-meta release necessary I think as just the docs were missing ☺️

[manniL]

@ricardogobbosouza Can you add a simple test (just modify one of the fixtures ;) to cover the change) please?

[pi0]

@manniL @ricardogobbosouza Shall we delegate example and tests to another PR or are you working on it? I don't want to make this enhancenemnt stalled.

[galvez]

This has gotten me before @ricardogobbosouza -- thanks!

[ricardogobbosouza]

Hi @pi0
I'm taking tests today.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.