Bugfix FXIOS-10599 ⁃ [Menu] - Menu functionalities not working (backport #23218)

mergify[bot] commented 2 days ago

:scroll: Tickets

:bulb: Description

Removed didFinish method call from onViewDismissed callback because if that is happening, self is nil inside the MainMenuCoordinator and navigateTo method will not pass the guard.

:pencil: Checklist

You have to check all boxes before merging

[x] Filled in the above information (tickets numbers and description of your work)
[x] Updated the PR name to follow our PR naming guidelines
[ ] Wrote unit tests and/or ensured the tests suite is passing
[ ] When working on UI, I checked and implemented accessibility (minimum Dynamic Text and VoiceOver)
[ ] If needed, I updated documentation / comments for complex code and public methods
[ ] If needed, added a backport comment (example @Mergifyio backport release/v120)

This is an automatic backport of pull request #23218 done by Mergify.

adudenamedruby commented 2 days ago

@DonalMe the fix for the QA menu issue

mobiletest-ci-bot commented 2 days ago

	Messages
:book:	Project coverage: 32.54%
:book:	Edited 1 files
:book:	Created 0 files

Client.app: Coverage: 30.67

File	Coverage
BrowserCoordinator.swift	70.83%	✅

Generated by :no_entry_sign: Danger Swift against 1c4a4ed37f0129591a03c3896d751bb889fcfb85

mozilla-mobile / firefox-ios