Skip to content

Conversation

atscott
Copy link
Contributor

@atscott atscott commented Oct 1, 2025

No description provided.

Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @atscott did you do some manual testing that everything still works, or should we do some testing?

@atscott
Copy link
Contributor Author

atscott commented Oct 1, 2025

@devversion I did some manual testing, clicking through the pages and expansion panels. I think this app is pretty safe as far as things go, since it’s pretty much serving static data and responding to click events to display new things. But I think it would be worthwhile to do a quick manual check on your end as well.

@AndrewKushnir
Copy link
Collaborator

FYI, did some manual testing as well, did not find any issues. Thanks for the update @atscott 👍 Merging the PR now...

@AndrewKushnir AndrewKushnir merged commit 669fd90 into angular:main Oct 2, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants