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

Fleet-Tutorial Milestones 4 #9

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Myestery
Copy link

@Myestery Myestery commented Jul 13, 2024

Fleet Tutorial for milestone 4
Added a page that describes retrieving boxes and using them
https://stackblitz.com/edit/typescript-atujdp?file=index.ts

Screenshot 2024-07-17 at 19 44 43

@Myestery
Copy link
Author

fix #8

@Myestery Myestery changed the title Fleet-Tutorial Milestones 4-11 Fleet-Tutorial Milestones 4 Jul 13, 2024
@Myestery Myestery marked this pull request as ready for review July 13, 2024 21:58
@Myestery Myestery marked this pull request as draft July 13, 2024 22:00
@Myestery
Copy link
Author

Myestery commented Jul 13, 2024

Hello, @arobsn @kii-dot
I was able to publish a page that should fix milestaone 4
However, for 5,6 I am a bit confused, does creating boxes mean calls to OutputBuilder ?
Or is there some info I can read up on it

@kii-dot
Copy link

kii-dot commented Jul 17, 2024

Can you add proper description so people can follow along?

@kii-dot
Copy link

kii-dot commented Jul 17, 2024

@Myestery good job. lets keep each pr for a single purpose. this being the usage of explorer api.
It looks good so far.

@arobsn would you be able to take a look too?

docs/use-of-boxes.md Outdated Show resolved Hide resolved
})();
```

An example of this implementation can be found [here](https://stackblitz.com/edit/typescript-atujdp?file=index.ts)
Copy link

Choose a reason for hiding this comment

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

i opened this link, and it didnt seem to log anything. Can you double check to ensure its working.

Adding a screen shot of it working to the description would help.

Copy link
Author

@Myestery Myestery Jul 17, 2024

Choose a reason for hiding this comment

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

Hi, I was able to interact just now and got the console output
Screenshot 2024-07-17 at 19 44 43

Do you have nautilus wallet installed?

@Myestery Myestery marked this pull request as ready for review July 17, 2024 18:40
@kii-dot
Copy link

kii-dot commented Jul 17, 2024

regarding 5 and 6, lets keep the conversation in the main thread at #8. so that we can refer back to it easily.

@Myestery
Copy link
Author

Hi @kii-dot
I have made the corrections
I think milestone 4 is done

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.

2 participants