Skip to content

Project overivew: Add permission check for edit button

Sascha Eggenberger requested to merge project-settings-permission-check into master

What does this MR do and why?

Project overview settings button:

  • Add permission check for edit button
  • Moved Project information title into the correct location
  • Some CSS class fixes

Screenshots or screen recordings

With permissions Without permissions
with_permission without_permission

How to set up and validate locally

  1. Enable FF echo "Feature.enable(:project_overview_reorg)" | rails c
  2. Check sidebar with permissions: http://gdk.test:3000/flightjs/Flight
  3. Check sidebar without permissions (e.g. logged out): http://gdk.test:3000/flightjs/Flight

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to ✂️ UX Paper Cuts 16.7 → Project page (#430244 - closed), #424396 (closed)

Edited by Sascha Eggenberger

Merge request reports

Loading