Skip to content

Conversation

@calintje
Copy link
Contributor

@calintje calintje commented Jan 31, 2025

  • Update example rust docs for fetch_positions_for_owner
  • Update navbar to include API
    • remove 'external-link' icon

Before

Screenshot 2025-01-31 at 16 21 21

After

Screenshot 2025-01-31 at 16 19 45

After comments

Screenshot 2025-01-31 at 20 29 24

Copy link
Member

@wjthieme wjthieme left a comment

Choose a reason for hiding this comment

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

Small comment. Otherwise LGTM

"@orca-so/whirlpools-rust": patch
---

Update example
Copy link
Member

Choose a reason for hiding this comment

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

Maybe a little more specific since these items will shop up in the changelogs of the SDK

@calintje calintje enabled auto-merge (squash) January 31, 2025 19:32
@calintje calintje merged commit 3bcb851 into main Jan 31, 2025
6 checks passed
@calintje calintje deleted the calintje/fix-example-rustdocs branch January 31, 2025 19:33
@github-actions github-actions bot mentioned this pull request Jan 31, 2025
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.

3 participants