Skip to content

Conversation

@clairep94
Copy link
Collaborator

@clairep94 clairep94 commented Oct 27, 2025

This was a left-over file I didn't do during the client/common folder migration, as I didn't understand how it worked. I understand it better now, so have migrated it

Changes:

  • client/common/icons

Notes:

Cherry-picked relevant commits from the giant rough-work PR here for easier review

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123
  • meets the standards outlined in the accessibility guidelines

@clairep94 clairep94 marked this pull request as ready for review October 27, 2025 03:14
@clairep94 clairep94 added the pr05 Grant Projects pr05 Grant Projects label Oct 27, 2025
@clairep94 clairep94 changed the title Pr05 finals migrate common icons Pr05 Typescript Migration #19: client/common/icons Oct 27, 2025
@clairep94 clairep94 changed the title Pr05 Typescript Migration #19: client/common/icons Pr05 Typescript Migration #19: client/common/icons Oct 27, 2025
@clairep94
Copy link
Collaborator Author

@raclim @khanniie super small one to wrap up client/common

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr05 Grant Projects pr05 Grant Projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant