Order of CP navbar items changing for non admin users

[krizzzly]

for some reason, non admin users dont have "dashboard" in first place after installing your plugin.

uncommenting: vendor/ether/tags/src/TagManager.php:156-158 helps

[ghost]

I guess there are two options:

It could check for permission:

if(Craft::$app->user->checkPermission('tag-manager')) {
    	Event::on(
    		Cp::class,
    		Cp::EVENT_REGISTER_CP_NAV_ITEMS,
    		[$this, 'onRegisterCpNavItems']
    	);
}

But imo a better solution would be that the plugin shouldn't mess with the menu order in the first place. It can't account for plugins or custom modules which order or customise the menu. (Granted.. Craft doesn't really give enough control on this)