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

fix: Add twitterHandle for a collectible #5007

Merged
merged 1 commit into from May 14, 2024

Conversation

Khushboo-dev-cpp
Copy link
Contributor

@Khushboo-dev-cpp Khushboo-dev-cpp commented Apr 2, 2024

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:

  • Something worth noting for reviewers.

Closes #

@status-im-auto
Copy link
Member

status-im-auto commented Apr 2, 2024

Jenkins Builds

Click to see older builds (133)
Commit #️⃣ Finished (UTC) Duration Platform Result
✖️ 47bcf95 #1 2024-04-02 20:59:15 ~2 min tests 📄log
✔️ 47bcf95 #1 2024-04-02 20:59:31 ~3 min linux 📦zip
✔️ 47bcf95 #1 2024-04-02 21:01:45 ~5 min android 📦aar
✔️ 47bcf95 #1 2024-04-02 21:03:29 ~7 min ios 📦zip
11e7269 #2 2024-04-05 16:26:06 ~16 sec linux 📄log
11e7269 #2 2024-04-05 16:27:01 ~1 min ios 📄log
11e7269 #2 2024-04-05 16:27:06 ~1 min android 📄log
✖️ 11e7269 #2 2024-04-05 16:28:16 ~2 min tests 📄log
76e70a5 #3 2024-04-08 08:34:54 ~14 sec linux 📄log
✖️ 76e70a5 #3 2024-04-08 08:35:56 ~1 min tests 📄log
76e70a5 #3 2024-04-08 08:36:00 ~1 min ios 📄log
76e70a5 #3 2024-04-08 08:36:01 ~1 min android 📄log
✖️ 5a2ee98 #4 2024-04-08 11:38:06 ~3 min tests 📄log
✔️ 5a2ee98 #4 2024-04-08 11:38:51 ~4 min ios 📦zip
✔️ 5a2ee98 #4 2024-04-08 11:39:02 ~4 min linux 📦zip
✔️ 5a2ee98 #4 2024-04-08 11:39:13 ~4 min android 📦aar
✖️ 857d8c4 #5 2024-04-09 16:51:55 ~2 min tests 📄log
✔️ 857d8c4 #5 2024-04-09 16:52:08 ~2 min linux 📦zip
✔️ 857d8c4 #5 2024-04-09 16:53:15 ~3 min ios 📦zip
✔️ 857d8c4 #5 2024-04-09 16:54:20 ~4 min android 📦aar
✔️ a6b0e5a #6 2024-04-09 16:57:28 ~3 min linux 📦zip
✔️ a6b0e5a #6 2024-04-09 16:58:14 ~4 min ios 📦zip
✖️ a6b0e5a #6 2024-04-09 16:58:49 ~5 min tests 📄log
✔️ a6b0e5a #6 2024-04-09 17:00:03 ~5 min android 📦aar
✔️ 6d5c89d #7 2024-04-09 17:00:10 ~2 min linux 📦zip
✖️ 6d5c89d #7 2024-04-09 17:00:18 ~1 min tests 📄log
✔️ 6d5c89d #7 2024-04-09 17:01:47 ~3 min ios 📦zip
✔️ 6d5c89d #7 2024-04-09 17:02:31 ~2 min android 📦aar
9e838a7 #8 2024-04-09 17:17:04 ~26 sec ios 📄log
9e838a7 #8 2024-04-09 17:17:48 ~1 min android 📄log
9e838a7 #8 2024-04-09 17:17:53 ~1 min linux 📄log
✖️ 9e838a7 #8 2024-04-09 17:24:26 ~7 min tests 📄log
✖️ 778861a #9 2024-04-09 17:30:06 ~1 min tests 📄log
✔️ 778861a #9 2024-04-09 17:31:12 ~2 min android 📦aar
✔️ 778861a #9 2024-04-09 17:31:53 ~3 min ios 📦zip
✔️ 778861a #9 2024-04-09 17:32:20 ~3 min linux 📦zip
✖️ dc21856 #10 2024-04-09 17:34:42 ~1 min tests 📄log
✔️ dc21856 #10 2024-04-09 17:36:26 ~3 min linux 📦zip
✔️ dc21856 #10 2024-04-09 17:36:37 ~3 min ios 📦zip
✔️ dc21856 #10 2024-04-09 17:38:03 ~4 min android 📦aar
✖️ 884a940 #11 2024-04-09 17:52:42 ~1 min tests 📄log
✔️ 884a940 #11 2024-04-09 17:53:49 ~2 min android 📦aar
✔️ 884a940 #11 2024-04-09 17:53:59 ~2 min linux 📦zip
✔️ 884a940 #11 2024-04-09 17:54:41 ~3 min ios 📦zip
✖️ 741f797 #12 2024-04-09 20:53:08 ~1 min tests 📄log
✔️ 741f797 #12 2024-04-09 20:54:23 ~2 min android 📦aar
✔️ 741f797 #12 2024-04-09 20:54:27 ~2 min linux 📦zip
✔️ 741f797 #12 2024-04-09 20:54:56 ~3 min ios 📦zip
✖️ e1e9347 #13 2024-04-09 21:19:38 ~1 min tests 📄log
✔️ e1e9347 #13 2024-04-09 21:20:38 ~2 min linux 📦zip
✔️ e1e9347 #13 2024-04-09 21:20:39 ~2 min android 📦aar
✔️ e1e9347 #13 2024-04-09 21:21:24 ~3 min ios 📦zip
✔️ f819a23 #14 2024-04-09 21:23:59 ~1 min android 📦aar
✔️ f819a23 #14 2024-04-09 21:24:15 ~2 min linux 📦zip
✔️ f819a23 #14 2024-04-09 21:25:04 ~2 min ios 📦zip
✖️ f819a23 #14 2024-04-09 22:01:39 ~39 min tests 📄log
✔️ 0f98ee5 #15 2024-04-09 21:54:30 ~2 min android 📦aar
✔️ 0f98ee5 #15 2024-04-09 21:54:40 ~2 min linux 📦zip
✔️ 0f98ee5 #15 2024-04-09 21:55:31 ~3 min ios 📦zip
✖️ 0f98ee5 #15 2024-04-09 22:41:18 ~39 min tests 📄log
✔️ 38828aa #16 2024-04-09 22:46:41 ~1 min linux 📦zip
✔️ 38828aa #16 2024-04-09 22:47:10 ~2 min android 📦aar
✔️ 38828aa #16 2024-04-09 22:48:31 ~3 min ios 📦zip
✖️ 38828aa #16 2024-04-09 23:24:05 ~39 min tests 📄log
✔️ 9d04dfb #18 2024-04-10 14:54:46 ~3 min android 📦aar
✔️ 9d04dfb #18 2024-04-10 14:57:42 ~6 min linux 📦zip
✔️ 9d04dfb #18 2024-04-10 15:16:04 ~13 min ios 📦zip
✔️ 9d04dfb #18 2024-04-10 15:33:29 ~41 min tests 📄log
✖️ fd25beb #19 2024-04-10 15:45:36 ~59 sec tests 📄log
✔️ fd25beb #19 2024-04-10 15:47:07 ~2 min android 📦aar
✔️ fd25beb #19 2024-04-10 15:47:12 ~2 min linux 📦zip
✔️ fd25beb #19 2024-04-10 15:48:05 ~3 min ios 📦zip
✖️ 98e29bb #20 2024-04-10 16:44:41 ~54 sec tests 📄log
✔️ 98e29bb #20 2024-04-10 16:45:21 ~1 min android 📦aar
✔️ 98e29bb #20 2024-04-10 16:46:12 ~2 min linux 📦zip
✔️ 98e29bb #20 2024-04-10 16:49:00 ~5 min ios 📦zip
✖️ 6085523 #21 2024-04-10 16:56:02 ~58 sec tests 📄log
✔️ 6085523 #21 2024-04-10 16:56:48 ~1 min android 📦aar
✔️ 6085523 #21 2024-04-10 16:57:36 ~2 min linux 📦zip
✔️ 6085523 #21 2024-04-10 16:58:31 ~3 min ios 📦zip
✔️ 92263e6 #22 2024-04-10 17:04:18 ~2 min linux 📦zip
✔️ 92263e6 #22 2024-04-10 17:04:29 ~2 min android 📦aar
✔️ 92263e6 #22 2024-04-10 17:04:42 ~3 min ios 📦zip
✔️ 92263e6 #22 2024-04-10 17:43:30 ~41 min tests 📄log
✔️ a4080c1 #23 2024-04-11 14:49:26 ~2 min android 📦aar
✔️ a4080c1 #23 2024-04-11 14:49:33 ~2 min linux 📦zip
✔️ a4080c1 #23 2024-04-11 14:50:37 ~3 min ios 📦zip
✔️ a4080c1 #23 2024-04-11 15:33:51 ~46 min tests 📄log
✔️ f4c2341 #24 2024-05-03 07:48:35 ~3 min linux 📦zip
✔️ f4c2341 #24 2024-05-03 07:49:27 ~4 min ios 📦zip
✔️ f4c2341 #24 2024-05-03 07:49:51 ~5 min android 📦aar
✖️ f4c2341 #24 2024-05-03 07:50:12 ~5 min tests 📄log
✔️ 8713bbb #25 2024-05-03 11:17:23 ~2 min linux 📦zip
✔️ 8713bbb #25 2024-05-03 11:18:29 ~3 min ios 📦zip
✔️ 8713bbb #25 2024-05-03 11:19:28 ~4 min android 📦aar
✖️ 8713bbb #25 2024-05-03 11:23:36 ~8 min tests 📄log
✔️ af48cf5 #26 2024-05-03 11:39:56 ~1 min android 📦aar
✔️ af48cf5 #26 2024-05-03 11:40:40 ~2 min linux 📦zip
✔️ af48cf5 #26 2024-05-03 11:41:21 ~3 min ios 📦zip
✖️ af48cf5 #26 2024-05-03 12:26:56 ~48 min tests 📄log
✖️ af48cf5 #27 2024-05-03 13:16:49 ~39 min tests 📄log
✔️ 4465324 #27 2024-05-03 13:15:37 ~4 min linux 📦zip
✔️ 4465324 #27 2024-05-03 13:16:20 ~4 min ios 📦zip
✔️ 4465324 #27 2024-05-03 13:17:04 ~5 min android 📦aar
✖️ 4465324 #28 2024-05-03 13:58:01 ~40 min tests 📄log
✔️ 93fe249 #28 2024-05-03 15:55:10 ~2 min android 📦aar
✖️ 93fe249 #29 2024-05-03 15:55:17 ~2 min tests 📄log
✔️ 93fe249 #28 2024-05-03 15:55:46 ~3 min ios 📦zip
✔️ 93fe249 #28 2024-05-03 15:56:33 ~3 min linux 📦zip
✔️ 7d189f5 #29 2024-05-03 15:57:08 ~1 min android 📦aar
✔️ 7d189f5 #29 2024-05-03 15:58:58 ~2 min linux 📦zip
✔️ 7d189f5 #29 2024-05-03 15:59:09 ~3 min ios 📦zip
✖️ 7d189f5 #30 2024-05-03 16:35:33 ~40 min tests 📄log
✔️ cbc8ff4 #30 2024-05-03 17:09:50 ~2 min android 📦aar
✔️ cbc8ff4 #30 2024-05-03 17:10:50 ~3 min ios 📦zip
✔️ cbc8ff4 #30 2024-05-03 17:11:29 ~3 min linux 📦zip
✖️ cbc8ff4 #31 2024-05-03 17:49:16 ~41 min tests 📄log
✔️ d39459e #31 2024-05-03 18:28:29 ~2 min linux 📦zip
✔️ d39459e #31 2024-05-03 18:28:44 ~2 min ios 📦zip
✔️ d39459e #31 2024-05-03 18:31:21 ~5 min android 📦aar
✖️ d39459e #32 2024-05-03 19:08:06 ~42 min tests 📄log
✔️ e5d4a07 #32 2024-05-03 20:05:01 ~1 min android 📦aar
✔️ e5d4a07 #32 2024-05-03 20:05:31 ~2 min linux 📦zip
✔️ e5d4a07 #32 2024-05-03 20:08:07 ~4 min ios 📦zip
✔️ e5d4a07 #33 2024-05-03 20:44:10 ~40 min tests 📄log
✔️ 2bee5b3 #33 2024-05-08 11:33:41 ~2 min linux 📦zip
✔️ 2bee5b3 #33 2024-05-08 11:34:23 ~3 min ios 📦zip
✔️ 2bee5b3 #33 2024-05-08 11:37:26 ~6 min android 📦aar
✔️ 2bee5b3 #34 2024-05-08 12:12:54 ~42 min tests 📄log
✔️ cc6867b #34 2024-05-13 22:12:51 ~4 min linux 📦zip
✔️ cc6867b #34 2024-05-13 22:13:32 ~4 min ios 📦zip
✔️ cc6867b #34 2024-05-13 22:13:45 ~5 min android 📦aar
✖️ cc6867b #35 2024-05-13 22:51:02 ~42 min tests 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ c887192 #35 2024-05-13 23:30:14 ~2 min linux 📦zip
✔️ c887192 #35 2024-05-13 23:32:28 ~4 min android 📦aar
✔️ c887192 #35 2024-05-13 23:32:46 ~5 min ios 📦zip
✔️ c887192 #36 2024-05-14 00:11:11 ~43 min tests 📄log

@Khushboo-dev-cpp Khushboo-dev-cpp force-pushed the AddTwitterHandleForACollectible branch 18 times, most recently from 9d04dfb to fd25beb Compare April 10, 2024 15:44
@Khushboo-dev-cpp Khushboo-dev-cpp marked this pull request as ready for review April 10, 2024 15:50
@Khushboo-dev-cpp Khushboo-dev-cpp force-pushed the AddTwitterHandleForACollectible branch 4 times, most recently from 92263e6 to a4080c1 Compare April 11, 2024 14:46
Copy link
Contributor

@stefandunca stefandunca left a 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.

services/wallet/activity/service_test.go Outdated Show resolved Hide resolved
services/wallet/collectibles/collection_data_db.go Outdated Show resolved Hide resolved
services/wallet/collectibles/collection_data_db.go Outdated Show resolved Hide resolved
services/wallet/collectibles/manager.go Outdated Show resolved Hide resolved
services/wallet/collectibles/service.go Outdated Show resolved Hide resolved
services/wallet/collectibles/collection_data_db.go Outdated Show resolved Hide resolved
services/wallet/api.go Outdated Show resolved Hide resolved
services/wallet/thirdparty/collectible_types.go Outdated Show resolved Hide resolved
services/wallet/thirdparty/rarible/types.go Outdated Show resolved Hide resolved
services/wallet/thirdparty/rarible/client.go Outdated Show resolved Hide resolved
services/wallet/collectibles/manager.go Outdated Show resolved Hide resolved
services/wallet/collectibles/collection_data_db.go Outdated Show resolved Hide resolved
services/wallet/api.go Outdated Show resolved Hide resolved
Copy link
Contributor

@dlipicar dlipicar left a 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

@Khushboo-dev-cpp Khushboo-dev-cpp force-pushed the AddTwitterHandleForACollectible branch 9 times, most recently from d39459e to e5d4a07 Compare May 3, 2024 20:03
Copy link
Contributor

@dlipicar dlipicar left a 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

services/wallet/api.go Outdated Show resolved Hide resolved
services/wallet/collectibles/manager.go Outdated Show resolved Hide resolved
services/wallet/collectibles/service.go Outdated Show resolved Hide resolved
services/wallet/collectibles/service.go Outdated Show resolved Hide resolved
services/wallet/collectibles/manager.go Outdated Show resolved Hide resolved
services/wallet/collectibles/collection_data_db.go Outdated Show resolved Hide resolved
@Khushboo-dev-cpp Khushboo-dev-cpp force-pushed the AddTwitterHandleForACollectible branch from e5d4a07 to 2bee5b3 Compare May 8, 2024 11:30
@Khushboo-dev-cpp Khushboo-dev-cpp force-pushed the AddTwitterHandleForACollectible branch from 2bee5b3 to cc6867b Compare May 13, 2024 22:08
@Khushboo-dev-cpp Khushboo-dev-cpp force-pushed the AddTwitterHandleForACollectible branch from cc6867b to c887192 Compare May 13, 2024 23:27
Copy link
Contributor

@dlipicar dlipicar left a comment

Choose a reason for hiding this comment

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

🎉

@Khushboo-dev-cpp Khushboo-dev-cpp merged commit 867cd1f into develop May 14, 2024
9 checks passed
@Khushboo-dev-cpp Khushboo-dev-cpp deleted the AddTwitterHandleForACollectible branch May 14, 2024 06:58
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.

None yet

4 participants