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

PubNub Swift Chat SDK 0.9.3 release #12

Merged
merged 5 commits into from
Dec 18, 2024
Merged

Conversation

jguz-pubnub
Copy link
Contributor

@jguz-pubnub jguz-pubnub commented Dec 18, 2024

refactor(project): run swiftformat to uplift the codebase
fix(custom-payloads): add missing initialization for reactionsActionName property
fix(channel): add missing completion: parameter when sending a text

@jguz-pubnub jguz-pubnub force-pushed the fix/channel-send-text branch from e71d567 to f8bf0d4 Compare December 18, 2024 10:39
meta: ["a": 123, "b": "someString"],
shouldStore: true,
mentionedUsers: mentionedUsers,
referencedChannels: referencedChannels,
usersToMention: nil,

Choose a reason for hiding this comment

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

I understand that we don't test deprecated method any more. Wouldn't be beneficial to have also test for deprecated method if we still offer it to clients?

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, I also added a test for the legacy method

@jguz-pubnub
Copy link
Contributor Author

@pubnub-release-bot release as 0.9.3-dev

@jguz-pubnub jguz-pubnub force-pushed the fix/channel-send-text branch from 6bf420d to d38e9b1 Compare December 18, 2024 13:26
@jguz-pubnub jguz-pubnub merged commit 9b666a0 into master Dec 18, 2024
6 checks passed
@jguz-pubnub jguz-pubnub deleted the fix/channel-send-text branch December 18, 2024 13:45
@pubnub-release-bot
Copy link

🚀 Release successfully completed 🚀

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