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

Entry widget and secure conversations v2 #1036

Draft
wants to merge 54 commits into
base: master
Choose a base branch
from

Conversation

rasmustautsglia
Copy link
Contributor

What was solved?

Release notes:

  • Feature
  • Ignore
  • Release notes (Is it clear from the description here?)
  • Migration guide (If changes are needed for integrator already using the SDK - what needs to be communicated? Add underneath please)

Additional info:

  • Is the feature sufficiently tested? All tests fixed? Necessary unit, acceptance, snapshots added? Check that at least new public classes & methods are covered with unit tests
  • Did you add logging beneficial for troubleshooting of customer issues?
  • Did you add new logging? We would like the logging between platforms to be similar. Refer to Logging from iOS SDKsThings to consider for newly added logs in Confluence for more information.

Screenshots:

@rasmustautsglia rasmustautsglia changed the title Feature/entry widget and secure conversations v2 Entry widget and secure conversations v2 Sep 9, 2024
@rasmustautsglia rasmustautsglia force-pushed the feature/entry-widget-and-secure-conversations-v2 branch 2 times, most recently from cd21e48 to fa640ff Compare September 23, 2024 18:11
@rasmustautsglia rasmustautsglia changed the base branch from master to development September 25, 2024 07:00
@ykyivskyi-gl ykyivskyi-gl force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from cfb864f to 17349d7 Compare October 8, 2024 08:44
@rasmustautsglia rasmustautsglia force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from cdf1e06 to cac42b4 Compare October 17, 2024 11:07
@igorkravchenko igorkravchenko force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from aed3878 to 0228f93 Compare October 17, 2024 15:49
Base automatically changed from development to master October 18, 2024 12:55
@igorkravchenko igorkravchenko force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from 7a5e1a8 to f4300ab Compare October 18, 2024 14:33
@andrews-moc
Copy link
Contributor

@rasmustautsglia I think the target branch should be changed to 'development', shouldn't it?

@igorkravchenko igorkravchenko force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from f4300ab to ef727e2 Compare October 21, 2024 07:22
This commit extend clone-snapshots-ci make command by running checkout.sh script, which tries to make checkout to the source branch in Snapshots if it exists, otherwise tries to make checkout to the destination branch if it exists, otherwise makes checkout to master branch

MOB-3716
@EgorovEI EgorovEI force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from ed763eb to 68ade5b Compare October 22, 2024 10:52
@EgorovEI EgorovEI force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from 68ade5b to e0349d0 Compare October 22, 2024 11:31
@EgorovEI EgorovEI changed the base branch from master to development October 22, 2024 11:56
@EgorovEI EgorovEI force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from e0349d0 to 6606af7 Compare October 22, 2024 12:06
Egor Egorov and others added 2 commits October 24, 2024 15:03
This commit fixes the case when OutgoingMessage stored in messagesSection did not include attachment object. It led to the situation when the attachment was missed if initial send message request failed.

MOB-3724
@igorkravchenko igorkravchenko force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from 9b2e7c1 to 14d76ce Compare November 1, 2024 14:18
Egor Egorov and others added 8 commits November 8, 2024 14:14
After adding AlertManager class configuring alert dialogs was broken. As a result for Audio, 1-way Video, 2-way Video offers the SDK always showed 2-way Video media upgrade dialog. That was only UI configuration issue, accepting the offer resulted in establishing right media type. This commit fixes alert configuration issue.

MOB-3768
This PR adds EntryWidget bottom sheet. In addition it adds the necessary
support files to be able to develop EntryWidget with SwiftUI.

MOB-3477
This PR adds the possibility to embed EntryWidget to a view. Also, as
was agreed upon within the team, there is no instance of EntryWidget
within Glia. EntryWidget instance will be returned from getEntryWidget
method

MOB-3479
Add Unified Customization to EntryWidget, as well as add missing UI
elements to accompany the customization options

MOB-3592
According to the design, new media type texts were added

MOB-3616
Egor Egorov and others added 19 commits November 11, 2024 15:22
This commit introduces:
 - new alert dialog called Leave Current Conversation;
 - style for the same dialog

MOB-3734
This commit:
 - Extends RemoteConfiguration.AlertStyle with negativeNeutralButton
 - fixes missed applying buttons alignment in alerts

MOB-3737
This PR checks for authentication status to determine if secure
messaging will be displayed in the available media type section in
Entry Widget

MOB-3658
Group accessibility media items. Add accessibility hints and types of items.

MOB-3697
Introduce new UI to indicate that sending of messages is unavailable on chat screen.

MOB-3726
Add unified customization send message unavailability indicator for chat screen.

MOB-3742
UISheetPresentation is by default favouring full screen cover in landscape mode,
due to lack of space. This however also disables drag to dismiss. This behavior
is overwritten in this PR and allows the sheet to be dismissed with click and with
drag.

MOB-3731
According to new SC 2.0 design we use 'send message unavailability' view instead of dialog. These changes remove dialog and show banner instead.

MOB-3739
Add toggling of SC bottom banner visibility based view model.

MOB-3719
Fix and add missed attributes.
Add a custom redacted placeholder that is not transparent by default.

MOB-3745
Due to the different nature of chat and call views, the behaviour of
alert dismissal worked way different. For Audio and Video engagements,
when confrmation dialog was declined, the engagement view was dismissed
alongside Entrywidget sheet. This was not intended. But for Chat, the dialog
dismissal didn't dismiss EntryWidget sheet. To align them all, the entry
widget is not dismissed when media type has been selected.

It also looks much better visually now with the animation.

MOB-3760
This PR covers EntryWidget with snapshot tests. This also includes
adding mocks for EntryWidget and EntryWidget Environment. In addition,
poweredBy was altered to allow scaling with font.

MOB-3545
static let was changed to static method and it didn't raise alarms during
rebasing
- Remove computed properties for title, subtitle, and hint from the MediaTypeItem.
- Add properties for the title, subtitle, and hint for each media type to the Entry Widget Style structures.
- Add functions that return the title, subtitle, and hint for the MediaTypeItem.
- Use functions from the Entry Widget Style structures in the EntryWidgetView to display strings.

MOB-3753
Introduce disabled state for ChatMessageEntry style for SC 2.0.

MOB-3748
Since message entry can have disabled state, file upload list, which is a part of message entry, also needs to have disabled state. These changes introduce such state. Styles use same colors for disabled state as for enabled for now, because we do not have a flow for disabled file upload.

MOB-3763
@EgorovEI EgorovEI force-pushed the feature/entry-widget-and-secure-conversations-v2 branch from ea35791 to 1ea3ecf Compare November 11, 2024 13:22
Base automatically changed from development to master November 12, 2024 09:19
ykyivskyi-gl and others added 8 commits November 12, 2024 12:34
According to new design for SC 2.0 buttons are not longer hidden but disabled.

MOB-3781
This commit introduces handling Live chat to Secure Conversations transferring

MOB-3630
This PR adds logs to EntryWidget, and aligning iOS with Android logs.
Because logger was added as a dependency to QueueMonitor, it was necessary
to move queueMonitor from Glia environment to Glia's class property due to
Logger being initialized during GliaSDK initialization. Because of that
some changes can be seen in tests as well.

MOB-3481
Update default global colors according to Mobile Color Tokens palette

MOB-3710
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants