Skip to content

[Protocol Dashboard] Fix user images not loading when MetaMask not connected; add loading state for OAuth modal [C-3694 C-3693] - #7258

Merged
nicoback2 merged 1 commit into
feature-dashboard-improvementsfrom
nkang--dwu-bugs-1
Jan 22, 2024
Merged

[Protocol Dashboard] Fix user images not loading when MetaMask not connected; add loading state for OAuth modal [C-3694 C-3693]#7258
nicoback2 merged 1 commit into
feature-dashboard-improvementsfrom
nkang--dwu-bugs-1

Conversation

@nicoback2

Copy link
Copy Markdown
Contributor

Description

How Has This Been Tested?

Tested locally against stage

Please describe the tests that you ran to verify your changes. Provide repro instructions & any configuration.

@nicoback2
nicoback2 requested review from a team and sliptype and removed request for a teamJanuary 20, 2024 00:04
@vercel

vercelBot commented Jan 20, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
NameStatusPreviewUpdated (UTC)
audius-web-ssr⬜️ Ignored (Inspect)Jan 20, 2024 0:04am

@nicoback2
nicoback2 requested a review from a teamJanuary 20, 2024 00:04
@@ -0,0 +1,43 @@
.container {

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Copy pasted this and LoadingSpinner from /embed

}
return instance
}
initAudiusSdk()

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

we don't need to init sdk in setup.ts anymore since we don't need the auth functions, so we can just init sdk here and get rid of the async stuff.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice

@sliptypesliptype left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

}
return instance
}
initAudiusSdk()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice

@nicoback2
nicoback2 merged commit a9b645c into feature-dashboard-improvementsJan 22, 2024
@nicoback2
nicoback2 deleted the nkang--dwu-bugs-1 branch January 22, 2024 19:16
Sign up for freeto 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

@nicoback2@sliptype@nicoback