-
Notifications
You must be signed in to change notification settings - Fork 119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
error "Application error" when clicking website menu -> "Account+" #2223
Comments
This has been fixed and released to production. |
really? I just approved this PR, is it no longer needed? #2240 |
whoops! I replied on the wrong ticket. This one is still needed. Disregard the above comment. |
* Also fixes an issue with nested anchor tags Fixes #2223
This has been validated in staging by clicking the collapsible sections in the mobile nav menu. Regression was performed to ensure that we're still properly firing the "accordion_opened" event for Simple Analytics. |
🤖 I have created a release *beep* *boop* --- ## [2.35.3](website-v2.35.2...website-v2.35.3) (2023-03-13) ### Bug Fixes * clarify docs when status is pinning ([#2242](#2242)) ([04ff58a](04ff58a)) * fix app failure when clicking 'Account +' on mobile ([#2240](#2240)) ([34e64ea](34e64ea)), closes [#2223](#2223) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Discovered While Testing:
Reproduction Steps:
![Screenshot 2023-02-23 at 3 05 07 PM](https://user-images.githubusercontent.com/171782/221051196-18306a9c-9d49-418b-93a2-fa3370cc8323.png)
* click the squiggly menu icon in top right. You should see menu. * click "Account+"Expected Behavior:
Actual Behavior:
The text was updated successfully, but these errors were encountered: