dgrigg / craft-migration-assistant

Migration Manager for Craft CMS
Other
27 stars 10 forks source link

'Create migration' button is visible for regular users in 'Globals' #31

Closed timbertens closed 3 years ago

timbertens commented 4 years ago

I never noticed it before, but one of our clients just got confused with the fact that there is a 'Create Migration' in 'Globals' for regular users. It is shown on the same spot as the 'Save' is normally, hence the confusion. I believe it is the intention that the functionality from 'Migration Assistant' in only visible / accessible for Admin account, not for regular users.

Screenshot 2020-11-25 at 12 13 12

Since regular users don't have access to migration, an error ('User is not permitted to perform this action') is thrown (which is normal) but doubles the confusion

Using versions: Craft Pro 3.5.15.1 Migration Assistant 3.2.3

dgrigg commented 4 years ago

@timbertens good catch. That should only be visible for users who have 'createContentMigrations' permission, same case as with the Entries/Categories etc.

timbertens commented 3 years ago

@dgrigg do you think it can be solved easily? I'm preparing a new release for these websites and would like to know if I can include it

dgrigg commented 3 years ago

@timbertens sorry been busy on a few things. I can make that update tomorrow.

timbertens commented 3 years ago

@dgrigg no problem, we are all over occupied .. ! Would be perfect, will plan the release this weekend in this case!

dgrigg commented 3 years ago

@timbertens release 3.2.4 is available with the update.