Skip to content

Add documentation for recommending dependency injection

Kushal Pandya requested to merge docs-fe-provide-inject into master

What does this MR do?

Following the discussions happened in gitlab-org/frontend/rfcs#81 (closed), this MR updates our Vue development guidelines to encourage using provide/inject instead of props when certain criteria is met.

How to setup and validate locally (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Merge request reports

Loading