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

feat: navigate to 'PSM' and 'Analytics' from the navigation bar #200

Merged
merged 2 commits into from
Sep 22, 2023

Conversation

anilhelvaci
Copy link
Contributor

Closes: #148 #157

Problem

Cross dapp connection needed to be improved

Work Done

Added two external links to navigate the user to PSM and Analytics pages that use the below links respectively

Here's the final look

Screen Shot 2023-09-15 at 11 35 46

Copy link
Member

@turadg turadg left a comment

Choose a reason for hiding this comment

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

Perfecto

@samsiegart
Copy link
Collaborator

Mind just running yarn lint:fix? That should fix the failing checks. We should add a CONTRIBUTORS.md or something to mention that...

@anilhelvaci
Copy link
Contributor Author

Mind just running yarn lint:fix? That should fix the failing checks. We should add a CONTRIBUTORS.md or something to mention that...

Done.

@samsiegart samsiegart merged commit a78123c into Agoric:main Sep 22, 2023
2 of 3 checks passed
@github-actions
Copy link

Network:
Commit: a78123c
Ref: refs/heads/main
IPFS v1 hash: bafybeiduigdw4cdgytqftzhbi446fcv4jod6uvjtqwfgkoafdkeinsj6uq
CF - DWeb - 4EVERLAND

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.

Vaults UI should include PSM link
3 participants