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

feat: 2312 pass invitation data to createNetwork saga and LAUNCH_COMMUNITY… #2438

Merged
merged 67 commits into from
Apr 25, 2024
Merged

feat: 2312 pass invitation data to createNetwork saga and LAUNCH_COMMUNITY… #2438

merged 67 commits into from
Apr 25, 2024

Conversation

EmiM
Copy link
Contributor

@EmiM EmiM commented Apr 15, 2024 •
edited
Loading

… payload

Pull Request Checklist

  • I have linked this PR to a related GitHub issue.
  • I have added a description of the change (and Github issue number, if any) to the root CHANGELOG.md.

(Optional) Mobile checklist

Please ensure you completed the following checks if you did any changes to the mobile package:

  • I have run e2e tests for mobile
  • I have updated base screenshots for visual regression tests

EmiM added 30 commits March 5, 2024 16:21
@EmiM
refactor: adjust customProtocolSaga code to be similar to deepLinkSag…
28dc343 
…a; Move parsing code from main.ts to saga
@EmiM
fix: customProtocolSaga tests
b8aa768
@EmiM
Merge branch 'develop' into feature/2310
928a412
@EmiM
fix: tests; add missing file
1460d57
@EmiM
fix: main.ts test
b4e8f45
@EmiM
feat: handle old (psk, orbitdbIdentity, addresses) and new (cid, toke…
7d0b19a 
…n, serverAddress, inviterAddress) invitation link format
@EmiM
chore: adjust types
d386067
@EmiM
chore: add joinNetwork saga that gathers data for createNetwork; mock…
d5f7b66 
… DOWNLOAD_INVITE_DATA response
@EmiM
Merge branch 'develop' into feature/2310
67268d2
@EmiM
refactor: invitation link parsers
a4d674b
@EmiM
fix: deepLink saga tests
8cc6fd3
@EmiM
refactor: simplify deepLink and customProtocol sagas
b8203f9
@EmiM
fix: invitation code utils tests
7d75c3a
@EmiM
chore: add missing github workflow for running common package tests
d333cfe
@EmiM
refactor: invitation code tests
de8533c
@EmiM
fix: unit tests
4895b29
@EmiM
Merge branch 'develop' into feature/2310
3b5cf43
@EmiM
Merge branch 'develop' into feature/2310
54b925c
@EmiM
feat: add one script for preparing AppImage for e2e tests
6e82f71
@EmiM
fix: 'copy app image for e2e' script
5f80ac1
@EmiM
fix: getting env file name in e2e tests
7471a24
@EmiM
chore: remove unused code responsible for locking invitation link for…
967665d 
…m on deep linking
@EmiM
chore: update changelog
9f84ffe
@EmiM
Merge branch 'develop' into feature/2310
bc2430d
@EmiM
Merge branch 'develop' into feature/2310
066a201
@EmiM
feat: add storage server proxy for downloading and uploading data fro…
3c6796d 
…m relay server
@EmiM
feat: joining with v2 invitation link
c6c9df4
@EmiM
feat: handle relay server connection errors
6e35699
@EmiM
test: add basic storage service proxy tests
7e6b4bc
@EmiM
feat: add basic server stored metadata validator
0963a59
@EmiM EmiM changed the title feat: pass invitation data to createNetwork saga and LAUNCH_COMMUNITY… feat: 2312 pass invitation data to createNetwork saga and LAUNCH_COMMUNITY… Apr 16, 2024
@EmiM EmiM marked this pull request as ready for review April 17, 2024 15:44
@EmiM EmiM requested review from leblowl and islathehut April 17, 2024 15:44
@EmiM EmiM changed the base branch from feature/2295 to develop April 22, 2024 15:18
This was referenced Apr 22, 2024
Closed
Open
leblowl
leblowl reviewed Apr 23, 2024
View reviewed changes
Copy link
Collaborator

@leblowl leblowl 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 to me overall, nice work.

packages/backend/src/nest/storageServerProxy/storageServerProxy.service.ts Outdated Show resolved Hide resolved
packages/desktop/src/renderer/components/CreateJoinCommunity/JoinCommunity/JoinCommunity.tsx Outdated Show resolved Hide resolved
packages/state-manager/src/sagas/appConnection/connection.selectors.ts Outdated Show resolved Hide resolved
packages/types/src/socket.ts Outdated Show resolved Hide resolved
packages/backend/src/nest/connections-manager/connections-manager.service.ts Show resolved Hide resolved
packages/backend/src/nest/libp2p/process-in-chunks.service.ts Show resolved Hide resolved
@EmiM EmiM merged commit de6f0cd into develop Apr 25, 2024
32 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leblowl leblowl leblowl approved these changes

@islathehut islathehut Awaiting requested review from islathehut

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@EmiM @leblowl