search

adonisjs / v6-docs

Documentation website for AdonisJS v6
38 stars 53 forks source link

Ioc container patch 1 #72

Closed Stancobridge closed 2 months ago

Stancobridge commented 3 months ago

While going through the documentation on IoC, in the section discussing how to define contextual dependencies, I realised that the documentation did not show where the drive is imported from. I just added the import line

thetutlage commented 3 months ago

There is no import path called @adonisjs/drive/services/main. The example has to be fixed by using driveManager.use instead of drive.use.

So can you please fix that?

Stancobridge commented 3 months ago

I got the import path from the container services documentation, under the container services vs Dependency injection section Container services vs. Dependency injection

If the drive path is not valid that means the documentation using Drive as an example in both IoC Container and Container Services page needs to be replaced with another example as Drive is not yet compatible with v6.

Stancobridge commented 3 months ago

@thetutlage This needs to be looked into as it is confusing engineers who are new to AdonisJS without knowing that some official packages are not yet compatible with AdonisJs v6.

thetutlage commented 2 months ago

The IoC Container docs goal is to explain the usage of container services in isolation. It is not a guide to get a list of available container services.