Skip to content

Comments

(K3 main) (NEW): Enoki Sample#90

Open
raphaelchiaml wants to merge 14 commits intomainfrom
feature/enoki-sample
Open

(K3 main) (NEW): Enoki Sample#90
raphaelchiaml wants to merge 14 commits intomainfrom
feature/enoki-sample

Conversation

@raphaelchiaml
Copy link
Collaborator

No description provided.

@raphaelchiaml
Copy link
Collaborator Author

sample display over at
https://enoki-example.vercel.app

@raphaelchiaml raphaelchiaml marked this pull request as ready for review January 28, 2026 13:38
Previously the enoki implementation only did sponsored transaction. But now it has google zklogin as well, which eliminates wallet extension popup if connected with zklogin.
Copy link
Collaborator

@domenicodev domenicodev left a comment

Choose a reason for hiding this comment

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

Great job Rapha,
just some small things to change:

Package managers

- Stick with one package manager and remove the **.lock** for the other

Folder organization

Let's organize it this way:

- contracts/
- - move/
- - - your_module_name/...
- frontend/
- - ...all the frontend here

So the root of this Module will just have frontend and contracts as top level folders

Move Package Management

First of all update your sui cli.
The new Package Management requires a Move.toml of a different format, basically you'll remove:

- addresses
- dev-addresses
- dev-dependencies

And replace move version:

from: 2024.beta
to: 2024

After finishing, make sure to run:

sui move build
and/or 
sui move test

Since if you're on devnet, the compiler will throw an error and ask you to put another entry in the Move.toml named "environments", and the respective values/properties to put in it;

@raphaelchiaml raphaelchiaml marked this pull request as draft February 13, 2026 06:15
@raphaelchiaml raphaelchiaml marked this pull request as ready for review February 16, 2026 06:08
Copy link
Collaborator

@domenicodev domenicodev left a comment

Choose a reason for hiding this comment

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

Great job @raphaelchiaml , just few considerations:

  1. I got that we use "use server" for serverfiles but could we separate those files from UI?
    From a react developer perspective and folder structure perspective it does not ideal. If it's a common practice in Nextjs then please ignore this.

  2. EDIT: skip this

  3. We need to update the SDK to the latest, which will use gRPC; feel free to reach out to me for a quick onboarding on it if necessary so you can migrate it quickly

@domenicodev domenicodev changed the title Feature/enoki sample (K3 main) (NEW): Enoki Sample Feb 24, 2026
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.

5 participants