Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add a new tutorial from web app ui starter (vite framework) #318

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from
Draft

Add a new tutorial from web app ui starter (vite framework) #318

wants to merge 13 commits into from

Conversation

antho31
Copy link
Contributor

@antho31 antho31 commented Sep 25, 2023

Step-by-step tutorial on how to build a web app on top of the OKP4 blockchain.

Vite project with pieces of code to show how to connect a wallet to the OKP4 testnet, interact with smart contracts, decode transactions

From https://github.com/okp4/web-ui-boilerplates

@antho31 antho31 self-assigned this Sep 25, 2023
@antho31 antho31 added the documentation Improvements or additions to documentation label Sep 25, 2023
@fredericvilcot
Copy link
Contributor

@antho31 @ccamel
This PR is linked to the web app boilerplate, that is a private repository with code that must be first improved and then validated by product team / web team.
So, I am not confortable with the idea to publish this work on the whitepaper for now.

@antho31 antho31 mentioned this pull request Sep 25, 2023
Merged
ccamel
ccamel requested changes Sep 25, 2023
View reviewed changes
ccamel
ccamel requested changes Sep 25, 2023
View reviewed changes
@antho31 antho31 force-pushed the feat/vite-boilerplate-tuto branch from 0d383c9 to ad1c861 Compare September 25, 2023 12:42
@antho31 antho31 requested a review from ccamel September 25, 2023 12:57
ErikssonJoakim
ErikssonJoakim approved these changes Sep 26, 2023
View reviewed changes
Copy link

@ErikssonJoakim ErikssonJoakim left a comment

Choose a reason for hiding this comment

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

Looks great, I just have some suggestions concerning the introduction 😃 Great work

antho31 and others added 2 commits September 26, 2023 11:16
@antho31 @ErikssonJoakim
fix(docs): add missing third party in vite tuto intro
496fcea 
Co-authored-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
@antho31 @ErikssonJoakim
fix(docs): reformulate difficulties with UI for vite tuto
dc5a4fa 
Co-authored-by: Joakim Eriksson <75730728+ErikssonJoakim@users.noreply.github.com>
@antho31
Copy link
Contributor Author

antho31 commented Sep 26, 2023

Thanks @ErikssonJoakim for your feedback 🙏

@ccamel ccamel force-pushed the main branch 4 times, most recently from 777dbc2 to 54d7bdc Compare November 21, 2023 19:46
@ccamel ccamel marked this pull request as draft January 7, 2024 17:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@fredericvilcot fredericvilcot fredericvilcot approved these changes

@ErikssonJoakim ErikssonJoakim ErikssonJoakim approved these changes

@amimart amimart Awaiting requested review from amimart

@MdechampG MdechampG Awaiting requested review from MdechampG

@JulienMassonnat JulienMassonnat Awaiting requested review from JulienMassonnat

@ccamel ccamel Awaiting requested review from ccamel

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@antho31 @fredericvilcot @ccamel @ErikssonJoakim