Skip to content

fix(frontend): adapt apps to sails-js and react-hooks update#665

Merged
LouiseMedova merged 7 commits intomasterfrom
tm/update-hooks
Apr 6, 2026
Merged

fix(frontend): adapt apps to sails-js and react-hooks update#665
LouiseMedova merged 7 commits intomasterfrom
tm/update-hooks

Conversation

@MedovTimur
Copy link
Copy Markdown
Contributor

No description provided.

@MedovTimur MedovTimur requested a review from vraja-nayaka March 31, 2026 10:27
Comment thread frontend/templates/create-vara-app/.eslintrc Outdated
Comment thread frontend/templates/create-gear-app/vite.config.ts
@vraja-nayaka
Copy link
Copy Markdown
Collaborator

I think we should also add the IDL source files like we did in ZK Poker (frontend\apps\zk-poker\src\app\utils\sails) and keep them next to the files generated from them. This way, if the sails library version change, we can easily regenerate the lib code.

Comment thread frontend/templates/create-gear-app/vite.config.js Outdated
Comment thread frontend/packages/ez-transactions/package.json
@MedovTimur MedovTimur requested a review from vraja-nayaka April 1, 2026 07:03
const pokerFactoryProgram = usePokerFactoryProgram();

const { data: ptsProgramId } = useProgramQuery({
const { data: ptsProgramId } = useTypedProgramQuery({
Copy link
Copy Markdown
Collaborator

@vraja-nayaka vraja-nayaka Apr 2, 2026

Choose a reason for hiding this comment

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

I don't thik that we need useTypedProgramQuery at all
I just change this line back to useProgramQuery and get this:

ptsProgramId: `0x${string}` | undefined

So useProgramQuery allready infer the types

Comment thread frontend/apps/zk-poker/src/app/utils/sails/sails.ts
const program = usePokerProgram();

const { data, refetch, isFetching, error } = useProgramQuery({
const { data, refetch, isFetching, error } = useTypedProgramQuery({
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The same about other using of useTypedProgramQuery

Comment thread frontend/packages/hooks/vite.config.ts Outdated
@@ -0,0 +1,26 @@
import { resolve } from 'path';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why do we need this config?
We don't need to build this package for using it in dapps.

@MedovTimur
Copy link
Copy Markdown
Contributor Author

Migrated create-gear-app and create-vara-app templates from legacy .eslintrc to local flat eslint.config.js. This removes the config-mode conflict with the monorepo root ESLint setup and lets vite-plugin-checker use plain eslint again without wrapper scripts or legacy-mode workarounds.

@MedovTimur MedovTimur requested a review from vraja-nayaka April 3, 2026 05:27
@LouiseMedova LouiseMedova merged commit a1dd489 into master Apr 6, 2026
18 checks passed
@LouiseMedova LouiseMedova deleted the tm/update-hooks branch April 6, 2026 09:59
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