Skip to content

Conversation

@k-antipochkin
Copy link
Contributor

@k-antipochkin k-antipochkin commented Aug 29, 2025

User description

Description of Changes

Update storage directory helpers: optimoveURL() uses Application Support with Caches/temp fallback and temporaryURL() returns system temp; no directory creation.

Breaking Changes

  • None

Release Checklist

Prepare:

  • [ ] Detail any breaking changes. Breaking changes require a new major version number
  • [ ] Check pod lib lint passes
  • [ ] Update any relevant sections of the repository wiki pages on a branch

Bump versions in:

  • OptimoveCore.podspec

  • OptimoveNotificationServiceExtension.podspec

  • OptimoveSDK.podspec

  • OptimoveCore/Sources/Classes/Constants/SDKVersion.swift

  • [ ] README.md

  • CHANGELOG.md

  • Update major version numbers in wiki (basic integration + push guides)

Integration tests

T&T Only

  • Init SDK with only T&T credentials
  • Associate customer
  • Associate email
  • Track events

Mobile Only

  • Init SDK with all credentials
  • Track events
  • Associate customer (verify both backends)
  • Register for push
  • Opt-in for In-App
  • Send test push
  • Send test In-App
  • Receive / trigger deep link handler (In-App/Push)
  • Receive / trigger the content extension, render image and action buttons for push
  • Verify push opened handler

Deferred Deep Links

  • With app installed, trigger deep link handler
  • With app uninstalled, follow deep link, install test bundle, verify deep link read from Clipboard, trigger deep link handler

Combined

  • Track event for T&T, verify push received
  • Trigger scheduled campaign, verify push received
  • Trigger scheduled campaign, verify In-App received

Release:

  • Squash and merge to master
  • Delete branch once merged
  • Create tag from master matching chosen version
  • Run pod trunk push to publish to CocoaPods

Post Release:

  • Push wiki pages to master


Generated description

Below is a concise technical summary of the changes proposed in this PR:
Update the SDK's storage directory helper functions, optimoveURL() and temporaryURL(), to manage file system paths more robustly by utilizing standard application support and temporary directories.

TopicDetails
Storage Paths Update the optimoveURL() helper to prioritize the Application Support directory with Caches and system temporary directory fallbacks, and modify temporaryURL() to directly return the system's temporary directory, removing explicit directory creation.
Modified files (1)
  • OptimoveCore/Sources/Classes/Storage/FileManager+Optimove.swift
Latest Contributors(1)
UserCommitDate
eli_g@optimove.comrc-5.6.0-84December 07, 2023
Version Updates Update the SDK version to 6.2.5 across all relevant podspec files (OptimoveCore.podspec, OptimoveNotificationServiceExtension.podspec, OptimoveSDK.podspec) and the SDKVersion.swift constant, and record the change in CHANGELOG.md.
Modified files (5)
  • OptimoveSDK.podspec
  • OptimoveNotificationServiceExtension.podspec
  • OptimoveCore/Sources/Classes/Constants/SDKVersion.swift
  • OptimoveCore.podspec
  • CHANGELOG.md
Latest Contributors(2)
UserCommitDate
adamball1bump-version-numbers-f...July 24, 2025
konstantin_a@optimove.comEmbedded-related-fixesJune 11, 2025
This pull request is reviewed by Baz. Review like a pro on (Baz).

@pr-work-item-validator
Copy link

👋 Hey @k-antipochkin,

❗️ Work Item link check failed: There is no Work Item linked to this PR.

🧩 Please add the Work Item ID to the PR title or description using the "Edit" option in the format below:
AB#XXXXX - replace X with a valid Work Item ID.

🟢 Once added, the validation will automatically run again!

1 similar comment
@pr-work-item-validator
Copy link

👋 Hey @k-antipochkin,

❗️ Work Item link check failed: There is no Work Item linked to this PR.

🧩 Please add the Work Item ID to the PR title or description using the "Edit" option in the format below:
AB#XXXXX - replace X with a valid Work Item ID.

🟢 Once added, the validation will automatically run again!

@pr-work-item-validator
Copy link

👋 Hey @k-antipochkin,

❗️ Work Item link check failed: There is no Work Item linked to this PR.

🧩 Please add the Work Item ID to the PR title or description using the "Edit" option in the format below:
AB#XXXXX - replace X with a valid Work Item ID.

🟢 Once added, the validation will automatically run again!

1 similar comment
@pr-work-item-validator
Copy link

👋 Hey @k-antipochkin,

❗️ Work Item link check failed: There is no Work Item linked to this PR.

🧩 Please add the Work Item ID to the PR title or description using the "Edit" option in the format below:
AB#XXXXX - replace X with a valid Work Item ID.

🟢 Once added, the validation will automatically run again!

@pr-work-item-validator
Copy link

👋 Hey @k-antipochkin,

❗️ Work Item link check failed: There is no Work Item linked to this PR.

🧩 Please add the Work Item ID to the PR title or description using the "Edit" option in the format below:
AB#XXXXX - replace X with a valid Work Item ID.

🟢 Once added, the validation will automatically run again!

2 similar comments
@pr-work-item-validator
Copy link

👋 Hey @k-antipochkin,

❗️ Work Item link check failed: There is no Work Item linked to this PR.

🧩 Please add the Work Item ID to the PR title or description using the "Edit" option in the format below:
AB#XXXXX - replace X with a valid Work Item ID.

🟢 Once added, the validation will automatically run again!

@pr-work-item-validator
Copy link

👋 Hey @k-antipochkin,

❗️ Work Item link check failed: There is no Work Item linked to this PR.

🧩 Please add the Work Item ID to the PR title or description using the "Edit" option in the format below:
AB#XXXXX - replace X with a valid Work Item ID.

🟢 Once added, the validation will automatically run again!

@k-antipochkin k-antipochkin changed the title Fix/258150 low storage AB#258150 low storage Aug 29, 2025
@k-antipochkin k-antipochkin marked this pull request as ready for review August 29, 2025 20:02
@k-antipochkin k-antipochkin merged commit da7e04c into master Sep 11, 2025
2 checks passed
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.

3 participants