Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: add screencast and new commands into read.me #1234

Conversation

juandiegocv27
Copy link

Enhance README with Screencast and Additional Commands

  • Adding a video tutorial to help newcomers understand the project workflow, from setup to deployment.
  • Including troubleshooting tips for function calls failing due to insufficient gas fees.
  • Updating usage instructions with detailed commands for testing, building, and deploying NEAR smart contracts.

Changes

  1. Usage Instructions:

    • Added cargo test before cargo near build.
    • Included commands for cargo near, cargo near build, cargo near create-dev-account, cargo near deploy, and cargo install near-cli-rs.
    • Updated near command for contract selection and function calls.
  2. Video Tutorial:

    • Added a link to the video tutorial.
    • Noted that the video shows a function call failure due to insufficient gas fees.
  3. Troubleshooting Section:

    • Added tips for handling insufficient gas fees during function calls.
    • Provided examples for allocating sufficient gas and checking account balance.

related issue: issue #1219

@juandiegocv27
Copy link
Author

@frol ready for feedback!

Copy link
Collaborator

@frol frol left a comment

Choose a reason for hiding this comment

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

Thanks for your attempt, but the result is not engaging. We need someone who can record and edit a video with nice transitions, highlights, and at least captions + music or voice-over.

README.md Outdated
Comment on lines 262 to 264
Got it. Here is the entire section formatted as markdown:

```markdown
Copy link
Collaborator

Choose a reason for hiding this comment

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

chatgpt...

Copy link
Author

@juandiegocv27 juandiegocv27 Aug 6, 2024

Choose a reason for hiding this comment

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

what´s that?, im going to fix it

@juandiegocv27
Copy link
Author

Thanks for your attempt, but the result is not engaging. We need someone who can record and edit a video with nice transitions, highlights, and at least captions + music or voice-over.

Ok sure, let me fix it and create a more engaging video 🔥🫡

@Jemiiah
Copy link

Jemiiah commented Aug 6, 2024

@frol i would love to work on this if you allow it🌚

@frol
Copy link
Collaborator

frol commented Aug 12, 2024

@Jemiiah Oh, I missed your comment. If you are still interested in it, please, go ahead!

@Jemiiah
Copy link

Jemiiah commented Aug 12, 2024 via email

@juandiegocv27
Copy link
Author

juandiegocv27 commented Aug 13, 2024

@frol I´ll finish this issue before tommorrow morning (less than 16 hours), I just updated this branch, sorry for he delay

@juandiegocv27 juandiegocv27 requested a review from frol August 13, 2024 00:14
@codecov-commenter
Copy link

codecov-commenter commented Aug 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (master@4551e43). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1234   +/-   ##
=========================================
  Coverage          ?   79.56%           
=========================================
  Files             ?      102           
  Lines             ?    14606           
  Branches          ?        0           
=========================================
  Hits              ?    11621           
  Misses            ?     2985           
  Partials          ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@frol
Copy link
Collaborator

frol commented Aug 14, 2024

@juandiegocv27 Any updates on this front?

@juandiegocv27
Copy link
Author

juandiegocv27 commented Aug 14, 2024

@juandiegocv27 Any updates on this front?

im uploaading the video in youtube, finish the editing, in a couple of hours I'll put in the read.me, plus making sure everything is ok

@juandiegocv27
Copy link
Author

@frol finished and ready to merge, let me know

@juandiegocv27
Copy link
Author

@frol sorry for the delay, editing videos is not my strong? can you assign this issue to someone else please

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.

4 participants