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

initial chain monitor, cluster chains panel #1405

Merged
merged 10 commits into from
Feb 19, 2023
Merged

initial chain monitor, cluster chains panel #1405

merged 10 commits into from
Feb 19, 2023

Conversation

floating
Copy link
Owner

@floating floating commented Feb 3, 2023

No description provided.

@floating floating added WIP PRs that are still in progress and not ready for review or merging UX labels Feb 3, 2023
shell.openExternal(`${explorer}${hashPath}`)
if (hash) {
const hashPath = hash && `/tx/${hash}`
shell.openExternal(`${explorer}${hashPath}`)

Check notice

Code scanning / Electronegativity

Review the use of openExternal

Review the use of openExternal
shell.openExternal(`${explorer}${hashPath}`)
} else if (account) {
const accountPath = account && `/address/${account}`
shell.openExternal(`${explorer}${accountPath}`)

Check notice

Code scanning / Electronegativity

Review the use of openExternal

Review the use of openExternal
const accountPath = account && `/address/${account}`
shell.openExternal(`${explorer}${accountPath}`)
} else {
shell.openExternal(`${explorer}`)

Check notice

Code scanning / Electronegativity

Review the use of openExternal

Review the use of openExternal
@floating floating marked this pull request as ready for review February 12, 2023 06:05
@floating floating removed the WIP PRs that are still in progress and not ready for review or merging label Feb 15, 2023
@floating floating requested a review from mholtzman February 15, 2023 18:03
mholtzman
mholtzman previously approved these changes Feb 19, 2023
@floating floating merged commit 74dc31e into develop Feb 19, 2023
@mholtzman mholtzman deleted the chain-monitor branch February 19, 2023 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants