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.

Sign up for GitHub

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

Jump to bottom

[SDK-3824] Remove PresenceMessage-related static things in tree-shakable library #1429

Merged
merged 1 commit into from
Nov 21, 2023
Merged

[SDK-3824] Remove PresenceMessage-related static things in tree-shakable library #1429

merged 1 commit into from
Nov 21, 2023

Conversation

lawrence-forooghian
Copy link
Collaborator

@lawrence-forooghian lawrence-forooghian commented Aug 23, 2023 •
edited
Loading

Note: This PR is based on top of #1428; please review that one first.

This does the same thing for PresenceMessage as we did for Message in 601b46b.

Resolves #1427.

@github-actions github-actions bot temporarily deployed to staging/pull/1429/features August 23, 2023 13:34 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report August 23, 2023 13:35 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc August 23, 2023 13:35 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the 1393-realtime-presence-tree-shaking branch from 1931f45 to 73c9b5f Compare August 23, 2023 14:41
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from 99d427c to 3e8f32f Compare August 23, 2023 14:41
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features August 23, 2023 14:42 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report August 23, 2023 14:43 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc August 23, 2023 14:43 Inactive
@lawrence-forooghian lawrence-forooghian marked this pull request as ready for review August 23, 2023 19:09
@lawrence-forooghian lawrence-forooghian force-pushed the 1393-realtime-presence-tree-shaking branch from 73c9b5f to 39fa2cd Compare August 23, 2023 19:37
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from 3e8f32f to babb0f6 Compare August 23, 2023 19:37
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features August 23, 2023 19:37 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report August 23, 2023 19:38 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc August 23, 2023 19:38 Inactive
@lawrence-forooghian lawrence-forooghian mentioned this pull request Aug 31, 2023
Merged
@lawrence-forooghian lawrence-forooghian force-pushed the 1393-realtime-presence-tree-shaking branch from 39fa2cd to d364b86 Compare September 5, 2023 08:38
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from babb0f6 to f571734 Compare September 5, 2023 08:40
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features September 5, 2023 08:40 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report September 5, 2023 08:42 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc September 5, 2023 08:42 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the 1393-realtime-presence-tree-shaking branch from d364b86 to 674c059 Compare September 14, 2023 12:53
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from f571734 to 77dcc87 Compare September 14, 2023 12:53
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features September 14, 2023 12:53 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc September 14, 2023 12:54 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report September 14, 2023 12:54 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the 1393-realtime-presence-tree-shaking branch from 674c059 to 78f0a35 Compare October 26, 2023 16:43
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from 77dcc87 to 594da2f Compare October 26, 2023 16:44
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features October 26, 2023 16:44 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc October 26, 2023 16:45 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features October 26, 2023 17:25 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc October 26, 2023 17:26 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report October 26, 2023 17:26 Inactive
@lawrence-forooghian lawrence-forooghian marked this pull request as draft November 1, 2023 18:33
@lawrence-forooghian lawrence-forooghian force-pushed the 1393-realtime-presence-tree-shaking branch from c80ea79 to d0a9d7c Compare November 7, 2023 19:09
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from ca58b56 to 861d66b Compare November 7, 2023 20:29
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features November 7, 2023 20:30 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report November 7, 2023 20:30 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc November 7, 2023 20:30 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from 861d66b to 4b79846 Compare November 8, 2023 17:05
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features November 8, 2023 17:06 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc November 8, 2023 17:07 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report November 8, 2023 17:07 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from 4b79846 to 962a06d Compare November 8, 2023 17:18
@github-actions github-actions bot temporarily deployed to staging/pull/1429/features November 8, 2023 17:19 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/typedoc November 8, 2023 17:20 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1429/bundle-report November 8, 2023 17:20 Inactive
@lawrence-forooghian lawrence-forooghian marked this pull request as ready for review November 8, 2023 18:03
owenpearson
owenpearson approved these changes Nov 14, 2023
View reviewed changes
Copy link
Member

@owenpearson owenpearson left a comment

Choose a reason for hiding this comment

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

Looks good!

@lawrence-forooghian lawrence-forooghian force-pushed the 1393-realtime-presence-tree-shaking branch from d0a9d7c to 7cbd6fc Compare November 14, 2023 18:48
@lawrence-forooghian lawrence-forooghian force-pushed the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch from 962a06d to 0e722b9 Compare November 14, 2023 18:50
Base automatically changed from 1393-realtime-presence-tree-shaking to integration/v2 November 21, 2023 00:10
@lawrence-forooghian lawrence-forooghian merged commit 9e56a5c into integration/v2 Nov 21, 2023
@lawrence-forooghian lawrence-forooghian deleted the 1427-remove-static-PresenceMessage-in-tree-shakable-library branch November 21, 2023 00:13
@lawrence-forooghian lawrence-forooghian mentioned this pull request Nov 21, 2023
Closed
@VeskeR VeskeR mentioned this pull request Mar 1, 2024
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@owenpearson owenpearson owenpearson approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lawrence-forooghian @owenpearson