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

Images Album #3021

Merged
merged 13 commits into from
Dec 14, 2022
Merged

Images Album #3021

merged 13 commits into from
Dec 14, 2022

Conversation

OmarBasem
Copy link
Member

@OmarBasem OmarBasem commented Dec 13, 2022

This PR adds AlbumID field to messages in order to be able to group images sent together like a collage.

@status-github-bot
Copy link

Pull Request Checklist

  • Have you updated the documentation, if impacted (e.g. docs.status.im)?
  • Have you tested changes with mobile?
  • Have you tested changes with desktop?

@status-im-auto
Copy link
Member

status-im-auto commented Dec 13, 2022

Jenkins Builds

Click to see older builds (33)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 77652e2 #1 2022-12-13 06:29:18 ~1 min linux 📦zip
✔️ 77652e2 #1 2022-12-13 06:32:15 ~4 min android 📦aar
✔️ 77652e2 #1 2022-12-13 06:34:06 ~6 min ios 📦zip
✔️ 62d41d2 #2 2022-12-13 07:17:07 ~58 sec linux 📦zip
✔️ 62d41d2 #2 2022-12-13 07:19:40 ~3 min android 📦aar
✔️ 62d41d2 #2 2022-12-13 07:19:47 ~3 min ios 📦zip
✔️ fbaa9eb #3 2022-12-13 08:21:52 ~54 sec linux 📦zip
✔️ fbaa9eb #3 2022-12-13 08:24:16 ~3 min android 📦aar
✔️ fbaa9eb #3 2022-12-13 08:24:44 ~3 min ios 📦zip
✔️ 2ad8588 #4 2022-12-13 08:36:00 ~1 min linux 📦zip
✔️ 2ad8588 #4 2022-12-13 08:38:29 ~3 min ios 📦zip
✔️ 2ad8588 #4 2022-12-13 08:39:03 ~4 min android 📦aar
✔️ e71382c #5 2022-12-13 09:04:59 ~2 min ios 📦zip
✔️ e71382c #5 2022-12-13 09:05:33 ~3 min android 📦aar
✔️ e71382c #5 2022-12-13 09:05:36 ~3 min linux 📦zip
✔️ e142dff #6 2022-12-13 09:17:53 ~59 sec linux 📦zip
✔️ e142dff #6 2022-12-13 09:19:54 ~3 min ios 📦zip
✔️ e142dff #6 2022-12-13 09:20:43 ~3 min android 📦aar
✔️ 5e02945 #7 2022-12-13 09:19:17 ~57 sec linux 📦zip
✔️ 5e02945 #7 2022-12-13 09:23:05 ~3 min ios 📦zip
✔️ 5e02945 #7 2022-12-13 09:24:06 ~3 min android 📦aar
✔️ c6d254a #8 2022-12-13 09:42:38 ~1 min linux 📦zip
✔️ c6d254a #8 2022-12-13 09:45:03 ~3 min android 📦aar
✔️ c6d254a #8 2022-12-13 09:45:21 ~3 min ios 📦zip
✔️ 308fdea #9 2022-12-14 05:19:02 ~57 sec linux 📦zip
✔️ 308fdea #9 2022-12-14 05:21:35 ~3 min android 📦aar
✔️ 308fdea #9 2022-12-14 05:22:02 ~3 min ios 📦zip
✔️ bc582de #10 2022-12-14 05:32:08 ~55 sec linux 📦zip
✔️ bc582de #10 2022-12-14 05:34:15 ~3 min ios 📦zip
✔️ bc582de #10 2022-12-14 05:34:39 ~3 min android 📦aar
✔️ 7de184e #11 2022-12-14 05:53:32 ~57 sec linux 📦zip
✔️ 7de184e #11 2022-12-14 05:55:36 ~3 min ios 📦zip
✔️ 7de184e #11 2022-12-14 05:56:00 ~3 min android 📦aar
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 1544442 #12 2022-12-14 06:07:11 ~52 sec linux 📦zip
✔️ 1544442 #12 2022-12-14 06:09:18 ~3 min ios 📦zip
✔️ 1544442 #12 2022-12-14 06:09:51 ~3 min android 📦aar
✔️ 12a0ca7 #13 2022-12-14 06:42:50 ~54 sec linux 📦zip
✔️ 12a0ca7 #13 2022-12-14 06:45:08 ~3 min ios 📦zip
✔️ 12a0ca7 #13 2022-12-14 06:45:23 ~3 min android 📦aar

@OmarBasem OmarBasem marked this pull request as ready for review December 14, 2022 06:43
@OmarBasem OmarBasem self-assigned this Dec 14, 2022
@OmarBasem OmarBasem merged commit d03691c into develop Dec 14, 2022
@OmarBasem OmarBasem deleted the images-album branch December 14, 2022 12:25
@0x-r4bbit
Copy link
Member

@OmarBasem we have users reporting:

ERR 2022-12-14 16:13:22.100+01:00 rpc response error                         topics="rpc" tid=52873 file=core.nim:29 err="\nstatus-go error [methodName:chat_getChats, code:-32000, message:sql: Scan error on column index 18, name \"album_id\": converting NULL to string is unsupported ]\n"

Are we show this works with databases that have messages which don't have an album_id ?

@0x-r4bbit
Copy link
Member

I think we need to ensure that this works with existing databases that don't have an AlbumID by wrapping album_id in

COALESCE(album_id, "")

0x-r4bbit added a commit that referenced this pull request Dec 14, 2022
Otherwise databases with existing messages that don't have an `album_id`
will run into and `converting NULL to string is unsupported` error.

See: #3021 (comment)
@0x-r4bbit
Copy link
Member

Created #3023 accordingly

0x-r4bbit added a commit that referenced this pull request Dec 16, 2022
Otherwise databases with existing messages that don't have an `album_id`
will run into and `converting NULL to string is unsupported` error.

See: #3021 (comment)
0x-r4bbit added a commit that referenced this pull request Dec 16, 2022
Otherwise databases with existing messages that don't have an `album_id`
will run into and `converting NULL to string is unsupported` error.

See: #3021 (comment)
0x-r4bbit added a commit that referenced this pull request Dec 16, 2022
Otherwise databases with existing messages that don't have an `album_id`
will run into and `converting NULL to string is unsupported` error.

See: #3021 (comment)
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

5 participants