Skip to content

Fix: provide/inject in plugins.md #3230

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Fix: provide/inject in plugins.md #3230

wants to merge 3 commits into from

Conversation

leoelz
Copy link
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
Copy link

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.

@bencodezen bencodezen added the state: review PR issued and under review label May 2, 2025
Copy link
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
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