Skip to content

Fix: provide/inject in plugins.md#3230

Merged
bencodezen merged 3 commits intovuejs:mainfrom
leoelz:patch-1
May 14, 2025
Merged

Fix: provide/inject in plugins.md#3230
bencodezen merged 3 commits intovuejs:mainfrom
leoelz:patch-1

Conversation

@leoelz
Copy link
Copy Markdown
Contributor

@leoelz leoelz commented Apr 30, 2025

plugins allow us to use provide to provide a function or attribute plugin, users are able to inject the plugin function or attribute

Description of Problem

It says: Plugins also allow us to use inject to provide a function or attribute to the plugin's users... and the example shows that provide is used. Also after the example it says Plugin users will now be able to inject the plugin options...

Proposed Solution

Replace inject with provide

Additional Information

plugins allow us to use `provide` to provide a function or attribute
plugin users are able to `inject` the plugin function or attribute
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 30, 2025

Deploy Preview for vuejs ready!

Name Link
🔨 Latest commit 2baf521
🔍 Latest deploy log https://app.netlify.com/sites/vuejs/deploys/6817c5409fa9740008a6929c
😎 Deploy Preview https://deploy-preview-3230--vuejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Comment thread src/guide/reusability/plugins.md Outdated
@bencodezen bencodezen added the state: review PR issued and under review label May 2, 2025
Copy link
Copy Markdown
Member

@bencodezen bencodezen left a comment

Choose a reason for hiding this comment

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

I provided revised phrasing on the sentence to reduce duplication of the word provide. Let me know what you think!

leoelz and others added 2 commits May 4, 2025 16:49
@bencodezen
Copy link
Copy Markdown
Member

Thanks for the PR @leoelz!

@bencodezen bencodezen merged commit d1b3d3b into vuejs:main May 14, 2025
5 checks passed
@leoelz leoelz deleted the patch-1 branch May 15, 2025 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state: review PR issued and under review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants