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
fix: Add twitterHandle for a collectible #5007
Conversation
Jenkins BuildsClick to see older builds (133)
|
9d04dfb
to
fd25beb
Compare
92263e6
to
a4080c1
Compare
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.
Great job extending the collectibles with caching socials info. I left some questions and concerns inline, please help with clarifications or changes.
walletdatabase/migrations/sql/1712763613_add_collection_socials.up.sql
Outdated
Show resolved
Hide resolved
walletdatabase/migrations/sql/1712763613_add_collection_socials.up.sql
Outdated
Show resolved
Hide resolved
walletdatabase/migrations/sql/1712763613_add_collection_socials.up.sql
Outdated
Show resolved
Hide resolved
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.
massive work! please check my comments before merging. They're mostly to keep consistency with how the rest of the collectibles module works
d39459e
to
e5d4a07
Compare
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.
Awesome! Just some extra changes
e5d4a07
to
2bee5b3
Compare
2bee5b3
to
cc6867b
Compare
…witter handle from alchemy
cc6867b
to
c887192
Compare
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.
🎉
A short summary which serves as a squashed-commit message.
Adds new table to hold collection socials such website and twitter handle. We can only rely on Alchemy to provide this data as of now due to missing api in rarible and opensea requests limitation.
In case Alchemy is the default provider we add values in the collection_socials_cache db. And in case other providers are being used the socials will be fetched from alchmey async and a signal with filled data will be sent to the client.
A description to understand introduced changes without reading the code.
Important changes:
Closes #