Skip to content

Commit

Permalink
Merge branch 'feature' into RMCCX-7351_click
Browse files Browse the repository at this point in the history
  • Loading branch information
SoumenRautray authored Sep 21, 2024
2 parents c902192 + bbcbc52 commit 63ba707
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
1 change: 0 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
## Changelog

### Unreleased
- Features:
- Implement clickable image through CustomJson [RMCCX-7233]

### 8.4.0 (2024-09-17)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,6 @@ internal class FullViewPresenter: BaseViewPresenter, FullViewPresenterType, Erro
}
logImpression(type: .clickContent)
sendImpressions()

UIApplication.shared.open(uriToOpen, options: [:], completionHandler: { [view] success in
if !success, let view = view {
self.showURLError(view: view)
Expand Down
2 changes: 2 additions & 0 deletions Tests/Tests/ViewPresenterSpec.swift
Original file line number Diff line number Diff line change
Expand Up @@ -774,6 +774,7 @@ class ViewPresenterSpec: QuickSpec {
expect(view.wasDismissCalled).to(beFalse())
expect(impressionService.sentImpressions).to(beNil())
}

it("will do nothing if redirect URL is invalid") {
let campaignClickableImage = TestHelpers.generateCampaign(id: "ClickableImage",
hasImage: true,
Expand All @@ -784,6 +785,7 @@ class ViewPresenterSpec: QuickSpec {
expect(view.wasDismissCalled).to(beFalse())
expect(impressionService.sentImpressions).to(beNil())
}

it("will do nothing if clickableImage is empty") {
let campaignClickableImage = TestHelpers.generateCampaign(id: "ClickableImage",
hasImage: true,
Expand Down

0 comments on commit 63ba707

Please sign in to comment.