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.

Sign up for GitHub

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

Jump to bottom

Update index.md #896

Merged
merged 4 commits into from
Mar 21, 2022
Merged

Update index.md #896

merged 4 commits into from
Mar 21, 2022

Conversation

bwann52
Copy link
Contributor

@bwann52 bwann52 commented Mar 17, 2022

updated links to panzerdog and sowork, both are upcoming

@bwann52
Update index.md
89ae4c7 
updated links to panzerdog and sowork, both are upcoming
@bwann52 bwann52 requested a review from Clay-Mysten March 17, 2022 15:58
Clay-Mysten
Clay-Mysten suggested changes Mar 17, 2022
View reviewed changes
Copy link
Contributor

@Clay-Mysten Clay-Mysten left a comment

Choose a reason for hiding this comment

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

Let's add these links with the files themselves.

@gdanezis gdanezis added the Type: Documentation Improvements or additions to documentation label Mar 17, 2022
bwann52 added 2 commits March 18, 2022 16:21
@bwann52
Update index.md
c447e23 
Added intro and videos, linked to existing files.
@bwann52
Merge branch 'main' into bwann52-patch-1
2b33830
@Clay-Mysten Clay-Mysten self-requested a review March 19, 2022 17:48
Clay-Mysten
Clay-Mysten approved these changes Mar 19, 2022
View reviewed changes
Copy link
Contributor

@Clay-Mysten Clay-Mysten left a comment

Choose a reason for hiding this comment

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

I found a couple of small issues worth addressing. And we won't truly be able to test the YouTube embeddings until this gets to staging. Preview mode doesn't render them.

@Clay-Mysten
Copy link
Contributor

Bauer, Alonso also has the Panzerdogs content coming in:
#996

Want to add that link back or have him/me do it? If the latter, please merge this in. Thanks!

@Clay-Mysten
Update index.md
3c76260 
Adding Panzerdogs to Explore index
@Clay-Mysten
Copy link
Contributor

Bauer, Alonso also has the Panzerdogs content coming in: #996

Want to add that link back or have him/me do it? If the latter, please merge this in. Thanks!

I took care of this in my own commit

@Clay-Mysten Clay-Mysten merged commit dce71b1 into main Mar 21, 2022
@Clay-Mysten Clay-Mysten deleted the bwann52-patch-1 branch March 21, 2022 20:27
mwtian pushed a commit to mwtian/sui that referenced this pull request Sep 29, 2022
@huitseeker
chore(ci): allow triggering bench, nightly manually (MystenLabs#896)
85f8d8d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Clay-Mysten Clay-Mysten Clay-Mysten approved these changes

@econmysten econmysten Awaiting requested review from econmysten

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

Successfully merging this pull request may close these issues.
3 participants
@bwann52 @Clay-Mysten @gdanezis