Skip to content

Fix letter_opener with method check in web_transaction

Paul Slaughter requested to merge ps-fix-letter-opener into master

What does this MR do and why?

I noticed locally that rails/letter_opener wasn't working. It looks like !132095 (merged) added a method call to endpoint_id_for_action expecting that every controller extends ApplicationController. This isn't the case for rails dev controllers like letter_opener.

Screenshots or screen recordings

Before After
Screenshot_2023-09-20_at_6.17.58_PM Screenshot_2023-09-20_at_6.18.17_PM

How to set up and validate locally

  1. Start GDK locally.
  2. Visit /rails/letter_opener.

MR acceptance checklist

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

Merge request reports

Loading