Skip to content
This repository was archived by the owner on Feb 17, 2025. It is now read-only.

[baking-with-ledger] handles feedback#526

Merged
timothymcmackin merged 3 commits intotrilitech:mainfrom
spalmer25:palmer@functori@make-baking-with-ledger-clearer
Jan 28, 2025
Merged

[baking-with-ledger] handles feedback#526
timothymcmackin merged 3 commits intotrilitech:mainfrom
spalmer25:palmer@functori@make-baking-with-ledger-clearer

Conversation

@spalmer25
Copy link
Contributor

Some feedback on our Baking with Ledger tutorial has been received.
This PR deals with these comments.

  • The baker account set up was not clear ; the readers were lost and do not clearly see what they should do.
    The command to runs for it as been included.

@vercel
Copy link

vercel bot commented Jan 23, 2025

@spalmer25 is attempting to deploy a commit to the Trili Tech Team on Vercel.

A member of the Team first needs to authorize it.

@spalmer25 spalmer25 changed the title [test] include command from Step 2: Set up a baker account [baking-with-ledger] handles feedback Jan 23, 2025
Because readers were confused, the `Set up a baker account` step was no longer clear enough.
Because it seems bakers mostly have main key on their ledger
@spalmer25 spalmer25 force-pushed the palmer@functori@make-baking-with-ledger-clearer branch from 08078a1 to a1aabc9 Compare January 24, 2025 07:43
Co-authored-by: Ajinkya  <145996984+ajinkyaraj-23@users.noreply.github.com>
@spalmer25 spalmer25 force-pushed the palmer@functori@make-baking-with-ledger-clearer branch from b0532d8 to d437c35 Compare January 27, 2025 12:05
@spalmer25 spalmer25 requested a review from ajinkyasraj January 27, 2025 12:06
Copy link
Contributor

@ajinkyasraj ajinkyasraj left a comment

Choose a reason for hiding this comment

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

LGTM

@ajinkyasraj
Copy link
Contributor

@timothymcmackin Can you please review?

@spalmer25 spalmer25 marked this pull request as ready for review January 28, 2025 07:36
@ajinkyasraj
Copy link
Contributor

hi @timothymcmackin , Can we merge this branch now?

@timothymcmackin timothymcmackin merged commit 494004f into trilitech:main Jan 28, 2025
1 of 3 checks passed
@timothymcmackin
Copy link
Contributor

hi @timothymcmackin , Can we merge this branch now?

Yup, merged, thanks for the submission!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants