Skip to content

Move method into computed properties

Savas Vedova requested to merge 368613-fetch-expanded-data-on-toggle into master

What does this MR do and why?

Fixes a minor bug by moving the collapseButtonLabel method into computed properties. Previously this went unnoticed as the label was still somehow working 🤷

Screenshots or screen recordings

Before and after are equal:

image

How to set up and validate locally

  1. This feature is behind a feature flag. Enable :refactor_security_extension to turn this feature on.
echo "Feature.enable(:refactor_security_extension)" | rails c
  1. Visit an MR page with Security Scanning

The easiest way to populate data for testing is using this extension https://chrome.google.com/webstore/detail/mokku-mock-api-calls-seam/llflfcikklhgamfmnjkgpdadpmdplmji?hl=en and importing mokku-mocks-17-8-2022.json this json.

Once you have enabled the extension:

  1. Open the developer tools
  2. Find Mokku tab
  3. Click on import
  4. Import the file mentioned above

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 #368613 (closed)

Merge request reports