Skip to content

Conversation

@NalinDalal
Copy link

Fixes #issue-number

Changes:
migrated client/modules/Preview to typescript

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

…useSyncFormTranslations to handle null formRef
…med export; update useSyncFormTranslations FormlikeType to take generic
@NalinDalal
Copy link
Author

Hi @NalinDalal ! Yes, including those types (as dev dependencies) would allow typescript to spot any type-errors in the codebase's use of those libraries, as well as give intellisense and (possibly) inline docs for those usages, too.

At the moment, your PR declares those library variables in a way which means they'll be given type any, essentially disabling type-checking for calls to those libraries and for the subsequent use of return values from those calls.

Just a suggestion!

thanks for heads up, types added.

@NalinDalal
Copy link
Author

@clairep94 @nbogie any updates on pr, do i need to do some changes, or is it rtm??

clairep94 and others added 26 commits December 14, 2025 07:01
…odulesUser

Pr05 Typescript Migration processing#21: `client/modules/User`
…_yarn/min-document-2.19.1

Bump min-document from 2.19.0 to 2.19.1
Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 3.13.1 to 3.14.2.
- [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md)
- [Commits](nodeca/js-yaml@3.13.1...3.14.2)

---
updated-dependencies:
- dependency-name: js-yaml
  dependency-version: 3.14.2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
…_yarn/js-yaml-3.14.2

Bump js-yaml from 3.13.1 to 3.14.2
We recently [released p5.js 2.1.2](https://github.com/processing/p5.js/releases/tag/v2.1.2)! This adds it to the version picker.

I have verified that this pull request:

* [x] has no linting errors (`npm run lint`)
* [x] has no test errors (`npm run test`)
* [x] is from a uniquely-named feature branch and is up to date with the  `develop` branch.
* [x] is descriptively named and links to an issue number, i.e. `Fixes processing#123`
Bumps [nodemailer](https://github.com/nodemailer/nodemailer) from 6.7.3 to 7.0.11.
- [Release notes](https://github.com/nodemailer/nodemailer/releases)
- [Changelog](https://github.com/nodemailer/nodemailer/blob/master/CHANGELOG.md)
- [Commits](nodemailer/nodemailer@v6.7.3...v7.0.11)

---
updated-dependencies:
- dependency-name: nodemailer
  dependency-version: 7.0.11
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…_yarn/nodemailer-7.0.11

build(deps): bump nodemailer from 6.7.3 to 7.0.11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Typescript Migration Related to Typescript Migration after the initial pr05 grant 2025 migration work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants