fix(media): fix send and media. #121
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
marwan2232004
requested review from
Ahmed-Aladdiin,
Mo2Hefny and
Bishoywadea
Mo2Hefny
added a commit
that referenced
this pull request
Dec 29, 2024
* Feature:Chat UI (#44) * feat(chat): add chat tiles to home page * feat(chat): add chat screen skeleton * Test: Unit tests for the user repository and sign up (#45) * test(utils): test utils functions. * test(user): test user repository. * refactor(keys): add auth keys to keys file. * feat(auth): byPass Recaptcha. * feat(chat): add chat screen with needed models (#48) * feat(chat): add chat screen messages * fix(chat): Fix draft Null bug * Fix crop feature (#47) * fix crop bug in release version * BUG(profile-picture): fix rectangle ratio * fix(profile-picture): resolve comment on navigating between pages * feat(chat): Add new chat creation (#49) * style(home): Update the chat tiles to match Telegram * feat(chat): implement new chat creation * FEATURE: Messaging infrastructure (#50) * feat(msg): implement the chat local repository * feat: update the chat local repository to store the other users * feat: add a message type enum * feat: add socket io package * feat: create a socket service class * feat: create a socket event handler with a sending loop * feat: create a chat remote repository * feat: create a basic class for the events queue * feat: basic structure of the chats view model * feat(msg): create the chatting controller (manager) * config: auto generated files * feat: add chat id to the chat model * feat: add emit with achknowledgement to the socket service * feat: implement Chats view model method * feat: implement msg sending logic * feat(messages): add the 'delete' and 'edit' events * fix: remove the extra chatID * fix(test): fix formatTimestamp utils test (#51) * fix(test): fix formatTimestamp utils test * fix(utils): fix formatTimestamp function * fix(test): fix typo in utils test * FEATURE: Messaging infrastructure (#52) * feat: complete the strucute of messaging * feat: finish messageing infrastructut * feat(chat): Add search functionality (#54) * feat(chat): add chat search functionality * feat(chat): add chat search functionality * test(chat): add chat related tests (#55) * test(chat): Add chat tile tests * test(chat): Add message tile tests * test(chat): Add kmp string matching tests * test(chat): Add chat search related ids * Test keys (#56) * feature: use the Keys class for testing purposes * refactor: remove warning causing code * same * test: add project analyze for cleaner code * Feature/media chat UI (#59) * feat(media-chat): add voice message recording ui. * feat(media-chat): enhance audio quality and some changes to ui. * feat(media-chat): add audio,image,video chat ui. * fix(media-chat): recording bug. * Media-UI (#53) * Fix crop feature (#47) * fix crop bug in release version * BUG(profile-picture): fix rectangle ratio * fix(profile-picture): resolve comment on navigating between pages * feat(chat): Add new chat creation (#49) * style(home): Update the chat tiles to match Telegram * feat(chat): implement new chat creation * FEATURE: Messaging infrastructure (#50) * feat(msg): implement the chat local repository * feat: update the chat local repository to store the other users * feat: add a message type enum * feat: add socket io package * feat: create a socket service class * feat: create a socket event handler with a sending loop * feat: create a chat remote repository * feat: create a basic class for the events queue * feat: basic structure of the chats view model * feat(msg): create the chatting controller (manager) * config: auto generated files * feat: add chat id to the chat model * feat: add emit with achknowledgement to the socket service * feat: implement Chats view model method * feat: implement msg sending logic * feat(messages): add the 'delete' and 'edit' events * fix: remove the extra chatID * fix(test): fix formatTimestamp utils test (#51) * fix(test): fix formatTimestamp utils test * fix(utils): fix formatTimestamp function * fix(test): fix typo in utils test * FEATURE: Messaging infrastructure (#52) * feat: complete the strucute of messaging * feat: finish messageing infrastructut --------- Co-authored-by: -NoName <[email protected]> Co-authored-by: Moamen Hefny <[email protected]> Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> * feat(media-chat): add message content classes. * feat(media-chat): update mock service. * feat(chat): fetch chats from server. * fix(chat): merge errors. * fix(chat): merge errors. * fix(chat): integrate chat updates with system * fix(chat): integrate chat updates with chat creation --------- Co-authored-by: -NoName <[email protected]> Co-authored-by: Moamen Hefny <[email protected]> Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> Co-authored-by: Mo2Hefny <[email protected]> * feat(chat): integrate the chat creation with the backend (#60) * FEAT(Stories): Integrate stories back-end (#58) * feat(stories): itegrate add, remove, see my stories with backend * feat(sessions): adding LottieViewer for marawan * feat(stories): integrat getting others stories from the backend and adding keys for e2e testing * fix(stories): fixing static code warnings * fix(stories): fixing keys to match the conventions * feat(chat):fetch chats and last message info. (#61) * feat(chat):fetch chats and last message info. * fix(chat):merge error. --------- Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> * fix: remove dublicate package * feat(chat): add chat mute UI (#62) * feat(chat): add chat mute UI * FIX: delete dublicate package * FIX: missing patanthesis in chat_screen * Fix: merging problems * feature: show user name in groups instaed of his id --------- Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> Co-authored-by: Ahmed Aladdin <[email protected]> * FEATURES: replay - Pin - Forward UI (#64) * feat(reply): done reply UI * feat(forward-pin): done forward and pin UI * fix: add final fixes to the merge --------- Co-authored-by: Bishoywadea <[email protected]> * messageing (#66) * FEATURE: Send messages (#67) * messageing * send messageas * feat(reply): reply message UI (#68) * Feature/send media messages (#65) * feat(chat): send media messages locally. * feat(chat): upload and download media. * fix(chat): message tile widget test. * fix(chat): merge conflicts. * feat(chat): add mock for uploading media. * hotfix(chat): fix create chat and pin messages. * feat(chat): integrate draft with mock and backend (#69) * hotfix(chat): fix create chat and pin messages. * feat(chat): add drafts - Add drafts integration with the mock and the backend. - Fix the repetition of private chats between the same users. * fix(draft): resolved hidden send button after fetch --------- Co-authored-by: marwan2232004 <[email protected]> * fix(media): resolve media and audio messages * hotfix(chat): fix create chat and pin messages. * fix(chat): fix audio recording and sending logic. * fix(chat): fix create chat screen. * fix(chat): fix mock. * fix(chat): fix mock. * hotfix(chat): login bug. * feat(chat): add mute system for mock (#71) * feat(chat): add drafts - Add drafts integration with the mock and the backend. - Fix the repetition of private chats between the same users. * feat(chat): add mute system mock * fix: resolve import * Feature: msg integeration v2 (#72) * feature: recieve messages * semi working * change the socket service --------- Co-authored-by: Marwan Alhameedy <[email protected]> * hotfix(chat): merge syntax problems. * Refactor: chat UI (#73) * refact(chat): refactor recording widget and adding recording service. * refact(chat): media sending and recording button. * refact(chat): optimize chat utils function. * refact(chat): better code structure. * feat: add essential ui pages for old and new modules (#74) * Feature/stickers UI (#76) * feat(chat): add media messages (Stickers,Emojis, Gifs) and download from the server. * feat(chat): change audio ui. * feat(chat): block user integration. (#77) * feat(chat): block user integration. * refact(user): remove prints. * fix(user): blocked users are not retrieved correctly at login. * Update chat_remote_repository.dart add todos for missing attributes --------- Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> * fix(chat): Resolve messaging events issues * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * fix: resolve errors thrown due to late initialized promises (#79) * hotfix(user): refactor naming convention. * hotfix(user): subtext color fix. * hotfix(user): fix sending empty media when the upload media is false. * hotfix(user): saving the media on the cache instead of storage. * hotfix(user): blocked users are not retrieved correctly at get me. * feat(calls): add calls UI (#80) * fix: resolve errors thrown due to late initialized promises * feat(call): add voice call UI and signaler * chore: remove .g.dart files from remote * refact: update call screen and signaler --------- Co-authored-by: marwan2232004 <[email protected]> * feat(media): add caption screen to images. * fix(media): document name overflow . * feat(media): add file name while sending and receiving. * feat(media): separating between music and voice. * refactor(search-and-discover): remove unnecessary tabs. * feat(search-and-discover): adding repo and view model. * feat(search-and-discover): refactor message content. * fix(search-and-discover): sending empty message. * feat(search-and-discover): modify home remote repo. * feat(search-and-discover): modify home view model. * feat(search-and-discover): search UI. * feat(search-and-discover): add file name for generated voices. * Feature/fix phase1 features (#81) * fix: fix home page reload * feat(groups): make UI for group creation * feat(groups): upload group image * Update build.gradle * Update MainActivity.kt * FIX: Send and Receive messages (#82) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feat: voice calls UI and signaler (#83) * fix: resolve errors thrown due to late initialized promises * feat(call): add voice call UI and signaler * chore: remove .g.dart files from remote * refact: update call screen and signaler * fix: refactor settings menu in chat screen * feat: add more settings menu to chat info page --------- Co-authored-by: marwan2232004 <[email protected]> * feat(chat): add Pin, Reply, Edit, and Delete messages (#84) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feature: implement pin and reply * fix: very tall reply message * fix: dublicate recieved messages issue * feature: add extra measures for establishing socket connection * feat: delete a message * feat: make an entire new socket connection * feat: check delete msg success * feature: handle keyboard dismiss in chat screen * fix: having an expiered session * feature: edit a message * fix: complete merging Feature/groups (#86) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feature: implement pin and reply * fix: very tall reply message * fix: dublicate recieved messages issue * feature: add extra measures for establishing socket connection * feat: delete a message * feat: make an entire new socket connection * feat: check delete msg success * feature: handle keyboard dismiss in chat screen * fix: having an expiered session * feature: edit a message * fix: complete merging * feat(groups): create group and receive event * feat(groups): make UI for group edit screen * feat(groups): make groups features * Update build.gradle * Update MainActivity.kt --------- Co-authored-by: Ahmed Aladdin <[email protected]> Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> Revert "Merge branch 'develop' into feature/search-and-discover" This reverts commit 988535e, reversing changes made to 2cad58c. fix(search-and-discover): highlighting problem. Feature/new msg events (#90) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feature: implement pin and reply * fix: very tall reply message * fix: dublicate recieved messages issue * feature: add extra measures for establishing socket connection * feat: delete a message * feat: make an entire new socket connection * feat: check delete msg success * feature: handle keyboard dismiss in chat screen * fix: having an expiered session * feature: edit a message * fix: complete merging fix: merge errors v1 fix: merge errors v2 delete unused file fix: merge errors v3 fix: return messaging functionality feature: delete message delete from server fix(search-and-discover): download print fix. fix(search-and-discover): put media url when sending. fix(search-and-discover): media id. fix(search-and-discover): media file name. add todo * feat(static-analysis): setup static analysis with SonarQube with GH Badges (#91) * feat(calls): Integrate one-to-one calls (#93) * Feature/fix phase1 features (#81) * fix: fix home page reload * feat(groups): make UI for group creation * feat(groups): upload group image * Update build.gradle * Update MainActivity.kt * FIX: Send and Receive messages (#82) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feat: voice calls UI and signaler (#83) * fix: resolve errors thrown due to late initialized promises * feat(call): add voice call UI and signaler * chore: remove .g.dart files from remote * refact: update call screen and signaler * fix: refactor settings menu in chat screen * feat: add more settings menu to chat info page --------- Co-authored-by: marwan2232004 <[email protected]> * feat(chat): add Pin, Reply, Edit, and Delete messages (#84) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feature: implement pin and reply * fix: very tall reply message * fix: dublicate recieved messages issue * feature: add extra measures for establishing socket connection * feat: delete a message * feat: make an entire new socket connection * feat: check delete msg success * feature: handle keyboard dismiss in chat screen * fix: having an expiered session * feature: edit a message * fix: complete merging * feat(calls): add signaling flow correctly * feat(calls): add one-to-one voice call * test: add keys for tests --------- Co-authored-by: -NoName <[email protected]> Co-authored-by: Moamen Hefny <[email protected]> Co-authored-by: marwan2232004 <[email protected]> Co-authored-by: Mo2Hefny <[email protected]> * feat: add groups (#92) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * Feature/fix phase1 features (#81) * fix: fix home page reload * feat(groups): make UI for group creation * feat(groups): upload group image * Update build.gradle * Update MainActivity.kt * FIX: Send and Receive messages (#82) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feat: voice calls UI and signaler (#83) * fix: resolve errors thrown due to late initialized promises * feat(call): add voice call UI and signaler * chore: remove .g.dart files from remote * refact: update call screen and signaler * fix: refactor settings menu in chat screen * feat: add more settings menu to chat info page --------- Co-authored-by: marwan2232004 <[email protected]> * feature: implement pin and reply * fix: very tall reply message * fix: dublicate recieved messages issue * feature: add extra measures for establishing socket connection * feat: delete a message * feat: make an entire new socket connection * feat: check delete msg success * feature: handle keyboard dismiss in chat screen * fix: having an expiered session * feature: edit a message * fix: complete merging * feat(groups): create group and receive event * feat(groups): make UI for group edit screen * feat(chat): add Pin, Reply, Edit, and Delete messages (#84) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feature: implement pin and reply * fix: very tall reply message * fix: dublicate recieved messages issue * feature: add extra measures for establishing socket connection * feat: delete a message * feat: make an entire new socket connection * feat: check delete msg success * feature: handle keyboard dismiss in chat screen * fix: having an expiered session * feature: edit a message * fix: complete merging * feat(groups): make groups features * Update build.gradle * Update MainActivity.kt * fix(groups): fix merge conflict * fix(UI): make web adaptive view * complete the merging of the develop branch --------- Co-authored-by: -NoName <[email protected]> Co-authored-by: Moamen Hefny <[email protected]> Co-authored-by: marwan2232004 <[email protected]> Co-authored-by: Bishoywadea <[email protected]> * feat: add turn servers (#94) * feat(search-and-discover): backend integration. (#95) * fix(messaging): (#96) missing chatId attribute sending localId of the message * FEATURE: Admin Dashboard (#97) * fix(search-and-discover): audio filter type. * fix(admin): activated,deactivate and ban users UI. * feat(admin): access a group and allow filtering. * fix(chat UI): fix reload of the chatScreen. And wrong timestamp of messages (#98) * fix: general fixes v2 (#99) * add skeletonizer to the chats list * hide the keyboard on application start * feat: add media settings UI (#100) * Feature/links (#102) * feat(link): add highlighting links and open them. * fix(admin): handle fetching users when being an admin. * feat(groups): group member search. * feat: add group call initializing (#101) * Feature/encryption (#103) * fix(messaging): missing chatId attribute sending localId of the message * feature(chatting): encryption * correctly encrypt and decrypt --------- Co-authored-by: Marwan Alhameedy <[email protected]> * fix: resolved chat list redundant updating (#104) * feat(admin): suspend users integration. (#105) * Feature/media integration (#106) * feat(admin): print. * refact(media): convert to GIF. * fix(media): search bug. * FEATURE(MESSAGING): Forward single or multiple messages (#107) * feature(messaging): forward single and multiple messages * add ui for the forwarded messages * feat(admin): group filtering (#109) * feat(admin): filter groups. * feat(admin): hide in appropriate messages. * fix: resolve draft and username integration (#110) * fix: resolve past integration issues * perf: make draft update in realtime * Test/keys (#113) * fix(search): remove duplicates. * test(block): add block feature testing keys. * feat(admin): filter bad messages. * test(media): media keys. * feat(admin): group filter. * fix(media): fix send and media. * Feature/channels (#112) * Feature/fix phase1 features (#81) * fix: fix home page reload * feat(groups): make UI for group creation * feat(groups): upload group image * Update build.gradle * Update MainActivity.kt * FIX: Send and Receive messages (#82) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feat: voice calls UI and signaler (#83) * fix: resolve errors thrown due to late initialized promises * feat(call): add voice call UI and signaler * chore: remove .g.dart files from remote * refact: update call screen and signaler * fix: refactor settings menu in chat screen * feat: add more settings menu to chat info page --------- Co-authored-by: marwan2232004 <[email protected]> * fix(groups): make sockets for being added * fix(groups): make sockets * feat(channels): make basic channel creation * feat(channels): Ui for channel threads * feat(channels): implement UI for threads --------- Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> Co-authored-by: Moamen Hefny <[email protected]> Co-authored-by: marwan2232004 <[email protected]> * Old msgs (#114) * get old messages * faeture: add load more button * fix: complete merging * Use global keys for login (#115) * fix: remove old login screen (#117) * feat: Highlight Mentions in Incoming Messages (#116) * feature: handle mentions in incoming messages * refactor: remove print statements * perf: fix reloading of chat screen with draft (#118) * Feature/fix channels (#119) * Feature/fix phase1 features (#81) * fix: fix home page reload * feat(groups): make UI for group creation * feat(groups): upload group image * Update build.gradle * Update MainActivity.kt * FIX: Send and Receive messages (#82) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feat: voice calls UI and signaler (#83) * fix: resolve errors thrown due to late initialized promises * feat(call): add voice call UI and signaler * chore: remove .g.dart files from remote * refact: update call screen and signaler * fix: refactor settings menu in chat screen * feat: add more settings menu to chat info page --------- Co-authored-by: marwan2232004 <[email protected]> * fix(channels): fix threads logic --------- Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> Co-authored-by: Moamen Hefny <[email protected]> Co-authored-by: marwan2232004 <[email protected]> * Feature/fix channels (#120) * Feature/fix phase1 features (#81) * fix: fix home page reload * feat(groups): make UI for group creation * feat(groups): upload group image * Update build.gradle * Update MainActivity.kt * FIX: Send and Receive messages (#82) * feat: sockets auto reconnect v1 * fix: sockets auto reconnect v2 * undate * fix: missing function call * fix: messageing step 1 * fix: remove the chats map from the chatViewModel to insure consistency * feature: add missing attributes * feature: get missing attributes from the request * Untrack the generated files * refactor: add some organization to the chat screen * feature: clear all chats and users data on log out * fix: remove dublicate user name from msg * feat: receive messages correctly * feat: voice calls UI and signaler (#83) * fix: resolve errors thrown due to late initialized promises * feat(call): add voice call UI and signaler * chore: remove .g.dart files from remote * refact: update call screen and signaler * fix: refactor settings menu in chat screen * feat: add more settings menu to chat info page --------- Co-authored-by: marwan2232004 <[email protected]> * fix(channels): fix threads logic * fix(channels): fix routes conflict --------- Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> Co-authored-by: Moamen Hefny <[email protected]> Co-authored-by: marwan2232004 <[email protected]> * fix(media): fix send and media. (#121) * fix(media): fix send and media. * fix(media): fix send and media. --------- Co-authored-by: Marwan Alhameedy <[email protected]> Co-authored-by: -NoName <[email protected]> Co-authored-by: Ahmed Aladdin Mohamed <[email protected]> Co-authored-by: Ahmed Aladdin <[email protected]> Co-authored-by: Bishoywadea <[email protected]> Co-authored-by: Amir Kedis <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.