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

add ibc withdraw method #284

Merged
merged 16 commits into from
Nov 17, 2024
Merged

add ibc withdraw method #284

merged 16 commits into from
Nov 17, 2024

Conversation

smk762
Copy link
Contributor

@smk762 smk762 commented Jul 17, 2024

closes #283

I had this commit unpushed from earlier, so not 100% sure it is complete. Please advise if any issue while testing, or if you notice anything missing.

Copy link

Preview for merge commit: 543bc93, available on cloudflare at: https://0f9b4a2c.komodo-docs.pages.dev/en/docs/

smk762 and others added 2 commits July 17, 2024 09:06
…nks, formatted md content, checked presence of file and dirs based on sidebar, checked presence of h1 in every file, update Komodo DeFi Framework methods table, adds/updates preview images when base is main
Copy link

Preview for merge commit: b8ab087, available on cloudflare at: https://ef1b6bde.komodo-docs.pages.dev/en/docs/

Copy link

Preview for merge commit: 9a4e739, available on cloudflare at: https://f1cb7375.komodo-docs.pages.dev/en/docs/

Copy link

Preview for merge commit: 610011d, available on cloudflare at: https://9ca728bb.komodo-docs.pages.dev/en/docs/

Copy link

Preview for merge commit: b72a5e0, available on cloudflare at: https://e047cc43.komodo-docs.pages.dev/en/docs/

Copy link

Preview for merge commit: 1e3768f, available on cloudflare at: https://f77a3b23.komodo-docs.pages.dev/en/docs/

Copy link

Preview for merge commit: 2f580c6, available on cloudflare at: https://dbc36729.komodo-docs.pages.dev/en/docs/

Copy link
Collaborator

@AndrewDelaney AndrewDelaney Oct 29, 2024

Choose a reason for hiding this comment

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

I'm assuming these are meant to be links. They are not formatted correct. Correct format should be:
[label](link)

Copy link
Contributor Author

@smk762 smk762 Oct 31, 2024

Choose a reason for hiding this comment

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

@gcharang not sure what's happening here, but when running run_tests.sh, the links on this page change to insert a \ between the (text) and [link].

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Update, reason found #374 (comment) 🤦

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed in 6e404b7

Copy link
Collaborator

Choose a reason for hiding this comment

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

This page is the index page for the Komodo Wallet Web guides (at least according to the page hierarchy in the left panel navigation). If this is meant to apply to General Komodo Wallet guides, should we not put this in a different page folder on a higher level than the Komodo Wallet guides for the various platforms?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Please open an issue for this, as it is unrelated to this PR

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this may actually be a problem with this PR. This preview contains 2 User Guides (Web) links in the folder hierarchy in the side panel, and both are expanded when 1 is clicked:

image image

This doesn't seem to be an issue on the current user guide pages.

I think the IBC guides will need their own parent entry.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks, feels like a merge fail. I'll have a look

Copy link
Contributor Author

@smk762 smk762 Nov 13, 2024

Choose a reason for hiding this comment

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

This should be fixed now

@shamardy
Copy link

shamardy commented Nov 8, 2024

@onur-ozkan can you please give this another review if there is anything left?

Copy link
Member

@onur-ozkan onur-ozkan left a comment

Choose a reason for hiding this comment

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

LGTM other than one note:

Copy link

Preview for merge commit: ff34aae, available on cloudflare at: https://ba3a93f3.komodo-docs.pages.dev/en/docs/

Copy link

Preview for merge commit: 92337a6, available on cloudflare at: https://7981a20e.komodo-docs.pages.dev/en/docs/

Copy link

Preview for merge commit: a4b2a10, available on cloudflare at: https://66da9862.komodo-docs.pages.dev/en/docs/

Copy link

Preview for merge commit: eebb9f3, available on cloudflare at: https://7b1c30d3.komodo-docs.pages.dev/en/docs/

@gcharang gcharang merged commit 55af7bb into dev Nov 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge P1 Important task which needs to be completed soon v2.1.0-beta-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants