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
[MOB-5473] Embedded GA Release #736
Conversation
[MOB-5746] payload fields
Anelson/fix elements button action
wrap embedded messages payload
…reground-retrieval
[MOB-5969] stop when logged out
[MOB-5964] foreground retrieval
# Conflicts: # tests/unit-tests/InAppHelperTests.swift
[MOB-5845] auto fetch boundary
[MOB-5973] route rename
# Conflicts: # swift-sdk/IterableAPI.swift
Version 6.5.0 alpha1
[MOB-6132] manual override refresh
[MOB-7679] Updates OOTB constraints
Matching the android's OOTB behaviour where Title and body are top aligned instead of being in center when displayed in banner view
By keeping the height of the stackview that holds the button equal to the button's height.
Image to title space optimized
Title and description texts can now reach till the image start if they exist or till the container's end
[MOB-7733] doesn't fetch embedded messages by default, adds a config option
@@ -0,0 +1,183 @@ | |||
# UI Components provided by IterableSDK | |||
|
|||
## IterableBannerView for iOS |
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.
We will be providing the doc in the code for the first time like this. cc @bradumbaugh.
Do we want that approach?
Build and run your project to see the IterableBannerView in action on your device or simulator. | ||
|
||
## Video Preview | ||
<!-- ![Preview](/BannerViewDocuments/full_design_video.mp4) --> |
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.
An mp4 video within SDK? Not sure about this @davidtruong what do you think?
BannerViewDocuments/Property.jpg
Outdated
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.
Xib inspectables are also kind of removed now.. might have to remove this image
BannerViewDocuments/Sample3.jpg
Outdated
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 sample images to though..
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.
Hmm. not sure if we are using this file anywhere.. have to revisit
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.
This file has been visited previously during the last sprints. Looks good
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.
Overall looking good. Left some comments and todo's to be taken care of
[MOB-XXXX] updates changelog and removes banner view documents directory
Placing a `less than equal to` constraint so that when image is not present, the nil image can provide its width for stackview beside it
Image in banner view to have constant width of 100 if its available. If not, the flexible constraint view can shrink the size of imgView making it available to stackview when image is not present
…View [MOB-7748] iOS constraints fix for banner img view
# Conflicts: # README.md
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.
Looking good. 🤞
🔹 Jira Ticket(s)
✏️ Description
GA ready embedded build branch