Skip to content

Conversation

@jguz-pubnub
Copy link
Contributor

@jguz-pubnub jguz-pubnub commented Jun 12, 2025

chore: add Getting Started guides to PubNub.xcworkspace as example iOS applications
chore: update Fastfile and standardize its environment files

@pubnub-ops-terraform
Copy link

pubnub-ops-terraform commented Jun 12, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@jguz-pubnub jguz-pubnub changed the title Add getting started guides Add getting started guides to PubNub.xcworkspace Jun 12, 2025
{
"idiom" : "universal",
"platform" : "ios",
"size" : "1024x1024"

Choose a reason for hiding this comment

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

Shouldn't filename be added here like:

{
      "idiom" : "universal",
      "platform" : "ios",
      "size" : "1024x1024",
      "filename" : "your_app_icon.png" // <--- Missing filename
    },

Copy link
Contributor Author

@jguz-pubnub jguz-pubnub Jun 18, 2025

Choose a reason for hiding this comment

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

It is required if you would like to submit your app to the AppStore.
We don't need an app icon for a Getting Started guide showing how to integrate an SDK

"appearances" : [
{
"appearance" : "luminosity",
"value" : "tinted"

Choose a reason for hiding this comment

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

Is tinted valid value for appearance?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this file is automatically generated if you create a new Xcode project from scratch

@jguz-pubnub jguz-pubnub force-pushed the feat/getting-started-guides branch from a350363 to 66cb595 Compare June 20, 2025 18:30
@jguz-pubnub jguz-pubnub force-pushed the feat/getting-started-guides branch 2 times, most recently from 590c843 to 5b2b3e6 Compare June 22, 2025 20:27
@jguz-pubnub jguz-pubnub merged commit 0035e78 into master Jun 23, 2025
21 of 51 checks passed
@jguz-pubnub jguz-pubnub deleted the feat/getting-started-guides branch June 23, 2025 10:25
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.

4 participants