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

feat: make wio independent from presence #429

Merged
merged 4 commits into from
Nov 21, 2023

Conversation

Raspincel
Copy link
Contributor

With these changes, the only prerequisite to use Who Is Online will be activating the component itself, instead of depending on the participant also activating Presence. Thanks to that change, a good chunk of code was removed due to contrived or out of date logic, and a few changes were made to the already existing method of updating the participants list.

@SuperViz-Dev
Copy link
Contributor

Total Coverage: 91.13%

Coverage Report
File Branch Funcs Lines Uncovered Lines
src
   shims.d.ts 100% 100% 100%
src/common
src/common/types
   events.types.ts 100% 100% 100%
   meeting-colors.types.ts 100% 100% 100%
   participant.types.ts 100% 100% 100%
   realtime.types.ts 100% 100% 100%
   sdk-options.types.ts 100% 100% 100%
src/common/utils
   do-request.ts 100% 100% 87.50% 20
   index.ts 100% 100% 100%
   logger.ts 100% 100% 100%
   observable.ts 100% 100% 100%
   observer.ts 100% 100% 100%
   sleep.ts 0% 100% 100% 2
src/components
   index.ts 100% 100% 100%
   types.ts 100% 100% 100%
src/components/base
   index.ts 100% 100% 100%
src/components/comments
   index.ts 92% 97.06% 98.47% 124, 191, 445, 542, 558
   types.ts 100% 100% 100%
src/components/comments/canvas-pin-adapter
   index.ts 86.96% 96.30% 95.72% 231, 232, 233, 234, 235, 336, 338, 338, 343, 353, 389, 434, 448, 449
src/components/presence-mouse
   index.ts 88.89% 83.33% 89.19% 130, 131, 132, 142, 177, 178, 179, 180, 181
src/components/realtime
   index.ts 100% 100% 100%
   types.ts 100% 100% 100%
src/components/video
   index.ts 100% 100% 100%
src/components/who-is-online
   index.ts 100% 100% 100%
   types.ts 100% 100% 100%
src/core
   index.ts 89.47% 100% 88.24% 49, 50, 69, 73, 75, 87
src/core/launcher
   index.ts 94.12% 100% 100% 75
src/services
src/services/api
   index.ts 100% 86.67% 86.54% 104, 105, 106, 107, 94, 95, 97
src/services/auth-service
   index.ts 100% 100% 100%
src/services/browser
   index.ts 100% 100% 100%
src/services/config
   index.ts 100% 100% 100%
src/services/connection-status
   index.ts 100% 100% 100%
src/services/event-bus
   index.ts 100% 100% 100%
src/services/frame-brick-layer
   index.ts 100% 100% 100%
src/services/limits
   index.ts 75% 100% 100% 10
src/services/message-bridge
   index.ts 93.33% 100% 100% 80
src/services/pubsub
   index.ts 100% 90% 100%
src/services/realtime
   index.ts 100% 100% 100%
src/services/realtime/ably
   index.ts 56.36% 60.95% 70.88% 100, 1003, 1004, 104, 1044, 1045, 1056, 1060, 1064, 108, 1100, 1159, 116, 1168, 1170, 1170, 1171, 120, 121, 1269, 1273, 1274, 1275, 1309, 1310, 1314, 1320, 1320, 1321, 1324, 1326, 1327, 1328, 1331, 1335, 1335, 1337, 1338, 1342, 1342, 1342, 1344, 1348, 1352, 1362, 1364, 1370, 1376, 1376, 1377, 1382, 1383, 1386, 1389, 1389, 1390, 1391, 1393, 1394, 1395, 1398, 1402, 1402, 1404, 1405, 1409, 1409, 1409, 1411, 1419, 1419, 1421, 1425, 1425, 1426, 1429, 1431, 1443, 1444, 1454, 1456, 1466, 1468, 1468, 1469, 1472, 194, 279, 420, 422, 432, 445, 458, 458, 460, 502, 512, 513, 534, 535, 537, 538, 540, 540, 541, 555, 557, 560, 562, 572, 573, 574, 575, 576, 576, 577, 578, 590, 600, 602, 604, 606, 607, 607, 613, 613, 613, 613, 613, 614, 616, 618, 621, 621, 621, 621, 621, 622, 624, 625, 625, 625, 625, 625, 626, 627, 627, 627, 627, 627, 628, 629, 629, 630, 630, 630, 630, 635, 637, 640, 642, 642, 642, 642, 643, 645, 673, 674, 684, 726, 727, 756, 758, 759, 760, 760, 761, 764, 771, 771, 772, 772, 774, 774, 774, 777, 778, 781, 781, 782, 783, 786, 787, 789, 794, 894, 894, 895, 898, 900, 900, 900, 901, 903, 957, 959, 960, 962, 964, 966, 966, 966, 966, 967, 968, 969, 972, 973, 976, 979, 979, 979, 979, 980
src/services/realtime/base
   index.ts 100% 100% 100%
src/services/remote-config-service
   index.ts 100% 100% 100%
src/services/video-conference-manager
   index.ts 64.10% 86.11% 93.62% 113, 116, 117, 118, 159, 164, 196, 197, 202, 203, 204, 207, 207, 208, 210, 211, 288, 323, 324, 325, 378, 412, 523, 535, 578, 582
   types.ts 100% 100% 100%

@SuperViz-Dev
Copy link
Contributor

Total Coverage: 98.08%

Coverage Report
File Branch Funcs Lines Uncovered Lines
src
src/web-components
src/web-components/base
   index.ts 100% 100% 100%
src/web-components/base/styles
   icon-button.style.ts 100% 100% 100%
   index.ts 100% 100% 100%
   sv-hr.style.ts 100% 100% 100%
   typography.style.ts 100% 100% 100%
   variables.style.ts 100% 100% 100%
src/web-components/comments
   comments.ts 92.86% 87.50% 98.23% 44, 45, 61
   index.ts 100% 100% 100%
src/web-components/comments/components
   annotation-filter.ts 100% 100% 100%
   annotation-item.ts 100% 100% 100%
   annotation-pin.ts 88.89% 100% 100% 35
   annotation-resolved.ts 100% 100% 100%
   annotations.ts 100% 88.89% 90.10% 39, 40, 41, 42, 56, 57, 58, 74, 75, 76
   comment-input.ts 96.43% 94.12% 87.86% 100, 101, 102, 103, 104, 105, 106, 107, 108, 109, 110, 111, 112, 113, 114, 115, 116, 117, 118, 119, 120, 121, 122, 123, 55, 99
   comment-item.ts 100% 100% 100%
   content.ts 96% 81.82% 88.71% 104, 105, 35, 36, 39, 40, 41, 47, 48, 49, 63, 74, 75, 76, 77
   delete-comment-modal.ts 100% 100% 100%
   float-button.ts 100% 100% 96.15% 41, 61, 62
   index.ts 100% 100% 100%
   topbar.ts 80% 100% 89.80% 38, 39, 40, 41, 42, 43
   types.ts 100% 100% 100%
src/web-components/comments/css
   annotation-filter.style.ts 100% 100% 100%
   annotation-item.style.ts 100% 100% 100%
   annotation-pin.style.ts 100% 100% 100%
   annotation-resolved.style.ts 100% 100% 100%
   annotations.style.ts 100% 100% 100%
   comment-input.style.ts 100% 100% 100%
   comment-item.style.ts 100% 100% 100%
   comments.style.ts 100% 100% 100%
   content.style.ts 100% 100% 100%
   float-button.style.ts 100% 100% 100%
   index.ts 100% 100% 100%
   powered-by.style.ts 100% 100% 100%
   topbar.style.ts 100% 100% 100%
src/web-components/comments/utils
   watermark.ts 100% 100% 90.77% 17, 18, 19, 20, 21, 22
src/web-components/dropdown
   index.style.ts 100% 100% 100%
   index.ts 100% 100% 100%
src/web-components/hello-world
   index.ts 100% 100% 100%
src/web-components/icon
   index.ts 100% 100% 100%
src/web-components/modal
   index.ts 100% 100% 100%
   modal-container.ts 100% 100% 100%
   modal.ts 100% 100% 100%
src/web-components/modal/styles
   index.style.ts 100% 100% 100%
src/web-components/presence-mouse
   index.ts 71.43% 100% 97.20% 42, 43, 44, 59, 60, 74, 75
   styles.ts 100% 100% 100%
src/web-components/who-is-online
   index.ts 100% 100% 100%
   who-is-online.ts 100% 100% 100%
src/web-components/who-is-online/components
   dropdown.ts 100% 100% 100%
   index.ts 100% 100% 100%
   types.ts 100% 100% 100%
src/web-components/who-is-online/css
   dropdown.style.ts 100% 100% 100%
   index.ts 100% 100% 100%
   who-is-online-style.ts 100% 100% 100%

@SuperViz-Dev
Copy link
Contributor

Total Coverage: 91.13%

Coverage Report
File Branch Funcs Lines Uncovered Lines
src
   shims.d.ts 100% 100% 100%
src/common
src/common/types
   events.types.ts 100% 100% 100%
   meeting-colors.types.ts 100% 100% 100%
   participant.types.ts 100% 100% 100%
   realtime.types.ts 100% 100% 100%
   sdk-options.types.ts 100% 100% 100%
src/common/utils
   do-request.ts 100% 100% 87.50% 20
   index.ts 100% 100% 100%
   logger.ts 100% 100% 100%
   observable.ts 100% 100% 100%
   observer.ts 100% 100% 100%
   sleep.ts 0% 100% 100% 2
src/components
   index.ts 100% 100% 100%
   types.ts 100% 100% 100%
src/components/base
   index.ts 100% 100% 100%
src/components/comments
   index.ts 92% 97.06% 98.47% 124, 191, 445, 542, 558
   types.ts 100% 100% 100%
src/components/comments/canvas-pin-adapter
   index.ts 86.96% 96.30% 95.72% 231, 232, 233, 234, 235, 336, 338, 338, 343, 353, 389, 434, 448, 449
src/components/presence-mouse
   index.ts 88.89% 83.33% 89.19% 130, 131, 132, 142, 177, 178, 179, 180, 181
src/components/realtime
   index.ts 100% 100% 100%
   types.ts 100% 100% 100%
src/components/video
   index.ts 100% 100% 100%
src/components/who-is-online
   index.ts 100% 100% 100%
   types.ts 100% 100% 100%
src/core
   index.ts 89.47% 100% 88.24% 49, 50, 69, 73, 75, 87
src/core/launcher
   index.ts 94.12% 100% 100% 75
src/services
src/services/api
   index.ts 100% 86.67% 86.54% 104, 105, 106, 107, 94, 95, 97
src/services/auth-service
   index.ts 100% 100% 100%
src/services/browser
   index.ts 100% 100% 100%
src/services/config
   index.ts 100% 100% 100%
src/services/connection-status
   index.ts 100% 100% 100%
src/services/event-bus
   index.ts 100% 100% 100%
src/services/frame-brick-layer
   index.ts 100% 100% 100%
src/services/limits
   index.ts 75% 100% 100% 10
src/services/message-bridge
   index.ts 93.33% 100% 100% 80
src/services/pubsub
   index.ts 100% 90% 100%
src/services/realtime
   index.ts 100% 100% 100%
src/services/realtime/ably
   index.ts 56.36% 60.95% 70.88% 100, 1003, 1004, 104, 1044, 1045, 1056, 1060, 1064, 108, 1100, 1159, 116, 1168, 1170, 1170, 1171, 120, 121, 1269, 1273, 1274, 1275, 1309, 1310, 1314, 1320, 1320, 1321, 1324, 1326, 1327, 1328, 1331, 1335, 1335, 1337, 1338, 1342, 1342, 1342, 1344, 1348, 1352, 1362, 1364, 1370, 1376, 1376, 1377, 1382, 1383, 1386, 1389, 1389, 1390, 1391, 1393, 1394, 1395, 1398, 1402, 1402, 1404, 1405, 1409, 1409, 1409, 1411, 1419, 1419, 1421, 1425, 1425, 1426, 1429, 1431, 1443, 1444, 1454, 1456, 1466, 1468, 1468, 1469, 1472, 194, 279, 420, 422, 432, 445, 458, 458, 460, 502, 512, 513, 534, 535, 537, 538, 540, 540, 541, 555, 557, 560, 562, 572, 573, 574, 575, 576, 576, 577, 578, 590, 600, 602, 604, 606, 607, 607, 613, 613, 613, 613, 613, 614, 616, 618, 621, 621, 621, 621, 621, 622, 624, 625, 625, 625, 625, 625, 626, 627, 627, 627, 627, 627, 628, 629, 629, 630, 630, 630, 630, 635, 637, 640, 642, 642, 642, 642, 643, 645, 673, 674, 684, 726, 727, 756, 758, 759, 760, 760, 761, 764, 771, 771, 772, 772, 774, 774, 774, 777, 778, 781, 781, 782, 783, 786, 787, 789, 794, 894, 894, 895, 898, 900, 900, 900, 901, 903, 957, 959, 960, 962, 964, 966, 966, 966, 966, 967, 968, 969, 972, 973, 976, 979, 979, 979, 979, 980
src/services/realtime/base
   index.ts 100% 100% 100%
src/services/remote-config-service
   index.ts 100% 100% 100%
src/services/video-conference-manager
   index.ts 64.10% 86.11% 93.62% 113, 116, 117, 118, 159, 164, 196, 197, 202, 203, 204, 207, 207, 208, 210, 211, 288, 323, 324, 325, 378, 412, 523, 535, 578, 582
   types.ts 100% 100% 100%

@SuperViz-Dev
Copy link
Contributor

Total Coverage: 98.08%

Coverage Report
File Branch Funcs Lines Uncovered Lines
src
src/web-components
src/web-components/base
   index.ts 100% 100% 100%
src/web-components/base/styles
   icon-button.style.ts 100% 100% 100%
   index.ts 100% 100% 100%
   sv-hr.style.ts 100% 100% 100%
   typography.style.ts 100% 100% 100%
   variables.style.ts 100% 100% 100%
src/web-components/comments
   comments.ts 92.86% 87.50% 98.23% 44, 45, 61
   index.ts 100% 100% 100%
src/web-components/comments/components
   annotation-filter.ts 100% 100% 100%
   annotation-item.ts 100% 100% 100%
   annotation-pin.ts 88.89% 100% 100% 35
   annotation-resolved.ts 100% 100% 100%
   annotations.ts 100% 88.89% 90.10% 39, 40, 41, 42, 56, 57, 58, 74, 75, 76
   comment-input.ts 96.43% 94.12% 87.86% 100, 101, 102, 103, 104, 105, 106, 107, 108, 109, 110, 111, 112, 113, 114, 115, 116, 117, 118, 119, 120, 121, 122, 123, 55, 99
   comment-item.ts 100% 100% 100%
   content.ts 96% 81.82% 88.71% 104, 105, 35, 36, 39, 40, 41, 47, 48, 49, 63, 74, 75, 76, 77
   delete-comment-modal.ts 100% 100% 100%
   float-button.ts 100% 100% 96.15% 41, 61, 62
   index.ts 100% 100% 100%
   topbar.ts 80% 100% 89.80% 38, 39, 40, 41, 42, 43
   types.ts 100% 100% 100%
src/web-components/comments/css
   annotation-filter.style.ts 100% 100% 100%
   annotation-item.style.ts 100% 100% 100%
   annotation-pin.style.ts 100% 100% 100%
   annotation-resolved.style.ts 100% 100% 100%
   annotations.style.ts 100% 100% 100%
   comment-input.style.ts 100% 100% 100%
   comment-item.style.ts 100% 100% 100%
   comments.style.ts 100% 100% 100%
   content.style.ts 100% 100% 100%
   float-button.style.ts 100% 100% 100%
   index.ts 100% 100% 100%
   powered-by.style.ts 100% 100% 100%
   topbar.style.ts 100% 100% 100%
src/web-components/comments/utils
   watermark.ts 100% 100% 90.77% 17, 18, 19, 20, 21, 22
src/web-components/dropdown
   index.style.ts 100% 100% 100%
   index.ts 100% 100% 100%
src/web-components/hello-world
   index.ts 100% 100% 100%
src/web-components/icon
   index.ts 100% 100% 100%
src/web-components/modal
   index.ts 100% 100% 100%
   modal-container.ts 100% 100% 100%
   modal.ts 100% 100% 100%
src/web-components/modal/styles
   index.style.ts 100% 100% 100%
src/web-components/presence-mouse
   index.ts 71.43% 100% 97.20% 42, 43, 44, 59, 60, 74, 75
   styles.ts 100% 100% 100%
src/web-components/who-is-online
   index.ts 100% 100% 100%
   who-is-online.ts 100% 100% 100%
src/web-components/who-is-online/components
   dropdown.ts 100% 100% 100%
   index.ts 100% 100% 100%
   types.ts 100% 100% 100%
src/web-components/who-is-online/css
   dropdown.style.ts 100% 100% 100%
   index.ts 100% 100% 100%
   who-is-online-style.ts 100% 100% 100%

Copy link
Collaborator

@carlossantos74 carlossantos74 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@carlossantos74 carlossantos74 merged commit b13bede into lab Nov 21, 2023
3 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 21, 2023
…independent-from-presence

feat: make wio independent from presence
@SuperViz-Dev
Copy link
Contributor

🎉 This PR is included in version 5.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants