Skip to content

Conversation

@clairep94
Copy link
Collaborator

@clairep94clairep94 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

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

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

@khanniiekhanniie requested a review from raclimNovember 4, 2025 22:10
@raclimraclim merged commit f079530 into processing:developNov 24, 2025
1 check passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr05 Grant Projectspr05 Grant Projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@clairep94@khanniie@raclim