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: Update icon #234
feat: Update icon #234
Conversation
…ion into ia--monetization-flow
Extension builds preview
|
…n-extension into df--update-icon
if (walletAddress) { | ||
startMonetization({ requestId, walletAddress }) | ||
isTabMonetized({ value: true }) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should bulk send (for start, resumse, stop) all the link tags that are in the page. This way we can avoid sending multiple events. With the current approach we send two START_MONETIZATION
and two TAB_MONETIZED
when a new link tag is added.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor updates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The icon change is working for me correctly, approving.
One comment I have is regarding @raducristianpopa 's comment: #234 (comment)
it seems like the start/stop/resume events are still being fired once per tag, maybe this can be made into a separate issue?
Closes #79