Documentation "Overriding files" > "Voyager-Models" has lack or wrong information

Version information

Laravel: v8.49.0
Voyager: v1.x-dev aaee6ab
PHP: 7.4.14
Database: mySQL 5.7.32

Description

When i overide the MenuItem (but i guess the problem is the same with other files) i put in my AppServiceProvier.php

class AppServiceProvider extends ServiceProvider
{
    public function register()
    {
        Voyager::useModel('MenuItem', \App\Models\MenuItem::class);

and in my \App\Models\MenuItem.php

    public static function boot()
    {
        parent::boot();

        static::retrieved(function ($model) {
            dd('retrieved');
        });
    }

sadly, the static::retrieved is never triggered.

Steps To Reproduce

Then i go to the Admin panel > Tools > Menu Builder > admin .... builder

Expected behavior

output the dump "retrieved"

Additional context

but if i put Voyager::useModel('MenuItem', \App\Models\MenuItem::class); in the boot() method instead of the register(), then the static::retrieved event is triggered...

So i suggest to modify the documentation to tell to put Voyager::useModel in the boot method instead.

the ultimate goal is to be able to override MenuItem and be sure that the original $model->menu->removeMenuFromCache(); are still executed. Otherwise, we get only the menu from the cache and not from the databse.

i didn't wanted to do a PR because i have no idea if movig that Voyager::useModel from the register() to boot() would had a negative impact on the voyager package.

thedevdojo / voyager