Skip to content

Conversation

alero-awani
Copy link
Contributor

@alero-awani alero-awani commented Oct 17, 2024

Notes for Reviewers

This PR fixes #337

Also changed the embedded design in the Embedding Designs doc to a better design.

image

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Alero Awani <aleroawani18@gmail.com>
Copy link

netlify bot commented Oct 17, 2024

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit 5632542
🔍 Latest deploy log https://app.netlify.com/sites/bejewelled-pegasus-b0ce81/deploys/6718e89cd852a600084ffdbe
😎 Deploy Preview https://deploy-preview-393--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@fitzergerald fitzergerald left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@leecalcote leecalcote left a comment

Choose a reason for hiding this comment

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

Quality work, @alero-awani.

Is there an existing section with code snippet that needs to be replaced?

@leecalcote
Copy link
Member

Quality work, @alero-awani.

Is there an existing section with code snippet that needs to be replaced?

I thought that we had a yaml or json example in here somewhere. It can make sense to continue to have as a behind-the-scenes way of setting this animation (instead of using the mouse to do it), but we'll need to make sure the it is still set in the same way. I don't think that it is. Soooo, we'll either need to delete or update.

@vishalvivekm
Copy link
Contributor

@alero-awani
Thank you for your contribution!
Let's discuss this during the website call on Monday at 5:30 PM IST (7:00 AM CT).

Please add it as an agenda item to the meeting minutes.

@vishalvivekm
Copy link
Contributor

@alero-awani Thank you for your contribution! Let's discuss this during the website call on Monday at 5:30 PM IST (7:00 AM CT).

Please add it as an agenda item to the meeting minutes.

adding

@leecalcote
Copy link
Member

Quality work, @alero-awani.

Is there an existing section with code snippet that needs to be replaced?

@alero-awani
Copy link
Contributor Author

@leecalcote This embedded design needs to be replaced with a more complex design. I'll do that and include it in this PR.

Signed-off-by: Alero Awani <aleroawani18@gmail.com>
@leecalcote leecalcote merged commit deb342d into layer5io:master Oct 25, 2024
6 checks passed
leecalcote added a commit that referenced this pull request Jun 10, 2025
Add Marching Ants animation to docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants