Skip to content

Martinh/portal faqs #469

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
Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Martinh/portal faqs #469

wants to merge 10 commits into from

Conversation

martin0995
Copy link
Collaborator

Description

  • Adds a new Portal FAQ page to the documentation (hidden from main navigation)
  • How to use deep-linking with query parameters
  • Explanation and screenshot of the “Send to a wallet address” field
  • Includes parameter reference table and a real-world example

Checklist

  • Required - I have added a label to this PR 🏷️
  • Required - I have run my changes through Grammarly
  • If pages have been moved, I have created redirects in the wormhole-mkdocs repo

@martin0995 martin0995 added the A0 - New Content Pull request contains new content pages label Jun 16, 2025
@martin0995 martin0995 added the B0 - Needs Review Pull request is ready for review label Jun 16, 2025
@martin0995 martin0995 requested a review from Telucero as a code owner June 16, 2025 11:12
dawnkelly09
dawnkelly09 reviewed Jun 17, 2025
View reviewed changes
Copy link
Collaborator

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

One small suggested change and then tagged Erin on that formatting/styling question. Once those are squared away, this should be good to go.

martin0995 and others added 2 commits June 18, 2025 04:17
@martin0995 @dawnkelly09
Apply suggestions from code review
9ead476 
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
@martin0995
llm check
6019c41
@martin0995 martin0995 requested a review from dawnkelly09 June 18, 2025 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eshaben eshaben Awaiting requested review from eshaben eshaben is a code owner

@0xlukem 0xlukem Awaiting requested review from 0xlukem 0xlukem is a code owner

@Telucero Telucero Awaiting requested review from Telucero Telucero is a code owner

@dawnkelly09 dawnkelly09 Awaiting requested review from dawnkelly09 dawnkelly09 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@dawnkelly09 dawnkelly09

Labels
A0 - New Content Pull request contains new content pages B0 - Needs Review Pull request is ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@martin0995 @eshaben @dawnkelly09