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

[MOB-5473] Embedded GA Release #736

Merged
merged 315 commits into from Jan 30, 2024
Merged

[MOB-5473] Embedded GA Release #736

merged 315 commits into from Jan 30, 2024

Conversation

davidtruong
Copy link
Contributor

@davidtruong davidtruong commented Jan 22, 2024

🔹 Jira Ticket(s)

✏️ Description

GA ready embedded build branch

roninopf and others added 30 commits March 23, 2023 12:08
# Conflicts:
#	tests/unit-tests/InAppHelperTests.swift
# Conflicts:
#	swift-sdk/IterableAPI.swift
davidtruong and others added 10 commits January 23, 2024 11:06
[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
@davidtruong davidtruong changed the title Embedded no diff merge [MOB-5473] Embedded no diff merge Jan 29, 2024
@davidtruong davidtruong changed the title [MOB-5473] Embedded no diff merge [MOB-5473] Embedded GA Release Jan 29, 2024
@@ -0,0 +1,183 @@
# UI Components provided by IterableSDK

## IterableBannerView for iOS
Copy link
Member

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) -->
Copy link
Member

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?

Copy link
Member

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

Copy link
Member

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..

Copy link
Member

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

Copy link
Member

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

Copy link
Member

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

Evan Greer and others added 4 commits January 29, 2024 12:41
[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
evantk91 and others added 2 commits January 30, 2024 08:59
…View

[MOB-7748] iOS constraints fix for banner img view
Copy link
Member

@Ayyanchira Ayyanchira left a comment

Choose a reason for hiding this comment

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

Looking good. 🤞

@Ayyanchira Ayyanchira merged commit ee39be7 into master Jan 30, 2024
2 of 3 checks passed
@Ayyanchira Ayyanchira deleted the embedded-no-diff-merge branch January 30, 2024 20:16
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

9 participants