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

Alpha #1280

Merged
merged 373 commits into from
May 14, 2024
Merged

Alpha #1280

merged 373 commits into from
May 14, 2024

Conversation

HarshRajat
Copy link
Contributor

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

mohammeds1992 and others added 30 commits January 18, 2024 20:14
* ci(restapi): 🎉 cut beta release to restapi-v0.0.1-alpha.61

* ci(restapi): 🎉 cut beta release to restapi-v0.0.1-alpha.62

* ci(restapi): 🎉 cut release to restapi-v0.9.0

* ci(restapi): 🎉 cut beta release to restapi-v0.0.1-alpha.63

* fix: fix errors

* ci(restapi): 🎉 cut beta release to restapi-v0.0.1-alpha.64

* fix: changed stream type to any

* ci(uiweb): 🎉 cut beta release to uiweb-v0.0.1-alpha.16

* fix: minor change

* ci(uiweb): 🎉 cut beta release to uiweb-v0.0.1-alpha.17

* fix: minor fix

* ci(uiweb): 🎉 cut beta release to uiweb-v0.0.1-alpha.36

* ci(uiweb): 🎉 cut beta release to uiweb-v0.0.1-alpha.37

* fix: fixed pagination bug and added functions for preload

* fix: added review comments

---------

Co-authored-by: Mohammed S <[email protected]>
* fix: fixed minor limit bug

* fix: exposed chatParticipant
* Added initial chat sidebar

* fix: pagination done

* Working stream with hacks, working badges, working loading, partial loading, not working accept stream

* fix: added themisation and ens resolve

* fix: fixed a bug

* streaming tweaks

* streaming tweaks with connected in restapi

* fix: added search for chatPreviewList

* fix: added error codes

* frontend react demo adjustment and rebasing

* duplicate import fix

* fix: refatored code

* fix: addec chat profile and create group changes

* fix: changed chatSender to chatParticipant

* fix: fixed scroll when not needed

* fix: changed group size

* fix: stream not getting connected

* fix: tested socket events

* fix: fixed bugs

---------

Co-authored-by: harshrajat <[email protected]>
Co-authored-by: Pritipriya Singh <[email protected]>
Co-authored-by: KlausMikhaelson <[email protected]>
* feat(video-v2): add highlevel video class

* feat(video): add video stream

* fix(sendnotification): modify rules.access.data to be an object & code cleanup

* fix(video): remove signer from input, throw err if signer, decrypted pgp key  not found

* feat(video): add sendNotification calls in connect, disconnect methods

* fix(videonotificationrules): typo in VideoNotificationRules interface

* add: useStream.ts video-v2-example-app

* add: videoV2.tsx in video-v2-example-app

* feat(video stream): handle connect, retry internally from the SDK

* feat(video connect): remove the connect method from the videov2 SDK

* fix(videov2): create push signer instance to get chain id in initialize()

* fix(videov2-example): refactor react example component & remove useStream hook

* fix(video stream): add backwards compatibilty to rules object for older SDK versions

* fix(video stream): fix bug in rules object creation

* fix(video stream): call connect() for ApproveVideo event

* feat(example-video-1.5): video v2  example app  implementation

* feat(sdk-frontend): few ui changes

* fix(video-v2-example-app): initialisevdeo call after a call ends

* feat(video stream): add request, deny evennt handlers

* feat(sdk-frontend and sdk-frontend-react): added video-v2 example implementation

* fix: few fixes

* refactor: refator and remove console logs

* fix: fix import error

* feat(video): update param names for video.initialize()

* fix: param name changes

* chore: added few comments

* chore: ui toast added

* fix(video example): remove pushprotocol sdk direct dir reference for initVideoCallData

* fix(video example): import React in examples & update SDK version in sdk-frontend

* feat(video example): replace video events from CONSTANTS obj

* Edited video example to make it more versatile

* Update index.js

* fix: added layout for userProfile

* fix: added modal

* fix: added modal

* fix: added modal content

* fix: added theme for userprofile

* fix: fixed bugs in profile

---------

Co-authored-by: Madhur Gupta <[email protected]>
Co-authored-by: Siddesh Sankhya <[email protected]>
Co-authored-by: Siddesh <[email protected]>
Co-authored-by: harshrajat <[email protected]>
Co-authored-by: Harsh | Push <[email protected]>
mishramonalisha76 and others added 28 commits May 2, 2024 17:40
* fix: fixed build issue (#1256)

* fix: published new version
* fix: published new version

* fix: fixed placement of chat (#1260)

* fix: fixed placement of chat

* fix: fixed group issue

* fix: fixed group issue (#1261)
…ing-fixes

refactor(multiple files): fixes export related terminal warning and updates yarn.lock
…e, fixed align issue when sender text was small
@HarshRajat HarshRajat merged commit 5a43891 into main May 14, 2024
1 check passed
Copy link

In index.js:

  1. There's a missing closing brace '}' after the content property in the userAlice.chat.send function.
  2. The console.log statement is missing a closing parenthesis ')'.

In package.json (uiweb):

  1. The peerDependencies object is missing a closing brace '}'.

In ActionRequestBubble.tsx:

  1. The exported interface IActionRequestBubbleProps is missing a closing brace '}'.
  2. In the catch block, the error variable should be caught individually like 'catch (err: Error | any)', you can't combine Error and any like that.

In ChatViewList.tsx:

  1. The import of ENCRYPTION_KEYS and EncryptionMessage is missing in the imports section.
  2. There is a missing closing brace '}' after the invalidChat property in the IChatViewListInitialized interface.
  3. The CHAT_STATUS constant object should have a closing brace '}'.
  4. The implementation of the useEffect hook seems to be incomplete, there are several missing elements like getMessagesCall function and transformSteamMessage function.
  5. The return statement and useEffect cleanup function in the useEffect block are not inside the callback function.
  6. The if block for chatAcceptStream and participantRemoveStream need to be inside the useEffect callback function.
  7. The usage of updatedChatInfo inside the if block is not properly defined as it seems to be declared outside the block and used inside.
  8. The transformSteamMessage, setChatStatusText, scrollToBottom, onScroll functions are referenced but not defined in the associated file.
  9. The if block for chatRequestStream has a similar issue as chatAcceptStream.
  10. There's a missing brace at the end of the useEffect hook.

Please address the highlighted issues. If you need further clarification or assistance, feel free to ask.

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.

9 participants