dysath / seat-fitting

A Fitting/Doctrine module for SeAT
GNU General Public License v2.0
14 stars 22 forks source link

Adding fitting.create permission to role does not add the ability to add fittings #3

Closed ZarosTenjin closed 6 years ago

ZarosTenjin commented 6 years ago

with superuser: selection_015

Version info: seat v3beta7 seat-fitting: v3.0.1alpha

dysath commented 6 years ago

Thanks, I will look into it!

On Fri, May 11, 2018 at 6:16 AM, ZarosTenjin notifications@github.com wrote:

-

Problem: Adding the Fitting.create permission to a role in Seat v3 does not enable the ability to add a fitting to the list. The button does not appear. Currently only sueprusers can do so.

Expected: the permission fitting.create enables the "add Fitting"-Button

Screenshot/Proof: no superuser but with fitting.create: [image: selection_014] https://user-images.githubusercontent.com/17838944/39921855-c37d1efe-551c-11e8-9133-4bb7a3cd517c.png

with superuser: [image: selection_015] https://user-images.githubusercontent.com/17838944/39921880-e1078626-551c-11e8-88c7-c7033c31cc81.png

Version info: seat v3beta7 seat-fitting: v3.0.1alpha

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dysath/seat-fitting/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/AQF4jD0xiOBBKz1jd2a4pEykqTIN0tNAks5txXMkgaJpZM4T7TZK .

-- Ed Stafford

dysath commented 6 years ago

Found the issue, fixed, and pushed a new release.

You should be able to update and confirm it's working properly.

On Fri, May 11, 2018 at 10:13 AM, Ed Stafford ed.stafford@gmail.com wrote:

Thanks, I will look into it!

On Fri, May 11, 2018 at 6:16 AM, ZarosTenjin notifications@github.com wrote:

-

Problem: Adding the Fitting.create permission to a role in Seat v3 does not enable the ability to add a fitting to the list. The button does not appear. Currently only sueprusers can do so.

Expected: the permission fitting.create enables the "add Fitting"-Button

Screenshot/Proof: no superuser but with fitting.create: [image: selection_014] https://user-images.githubusercontent.com/17838944/39921855-c37d1efe-551c-11e8-9133-4bb7a3cd517c.png

with superuser: [image: selection_015] https://user-images.githubusercontent.com/17838944/39921880-e1078626-551c-11e8-88c7-c7033c31cc81.png

Version info: seat v3beta7 seat-fitting: v3.0.1alpha

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dysath/seat-fitting/issues/3, or mute the thread https://github.com/notifications/unsubscribe-auth/AQF4jD0xiOBBKz1jd2a4pEykqTIN0tNAks5txXMkgaJpZM4T7TZK .

-- Ed Stafford

-- Ed Stafford

ZarosTenjin commented 6 years ago

can confirm issue is solved.