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

bring FLEx lite poc into lexbox #811

Merged
merged 101 commits into from
May 28, 2024
Merged

bring FLEx lite poc into lexbox #811

merged 101 commits into from
May 28, 2024

Conversation

hahn-kev
Copy link
Collaborator

FLEx lite is the WeSay/LF replacement, right now it's just an aspnet core server that when run launches the users browser, we'll probably turn this into some kind of electron app in the future.

This also adds a CrdtCommits table to lexbox to facilitate syncing data. This should be considered experimental and subjet to change.There's also no auth right now, so this won't work as is, we need to get the openId stuff figured out and merged in before that will work. However FLEx lite is designed with offline in mind, so it'll work fine if it can't contact lexbox.

right now all the local webapp code lives in the backend folder, not sure that's the correct place for it. Thoughts?

hahn-kev and others added 30 commits April 17, 2024 09:49
…L has been returned from a call to a COM component. The error code is E_FAIL, or EFail, or 0x80004005.` error by disabling build options in debug mode
Copy link

github-actions bot commented May 22, 2024

UI unit Tests

11 tests   11 ✅  0s ⏱️
 3 suites   0 💤
 1 files     0 ❌

Results for commit ff718d8.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 22, 2024

C# Unit Tests

57 tests   57 ✅  11s ⏱️
11 suites   0 💤
 2 files     0 ❌

Results for commit ff718d8.

♻️ This comment has been updated with latest results.

backend/LcmCrdt/CrdtLexboxApi.cs Outdated Show resolved Hide resolved
backend/LocalWebApp/BackgroundSyncService.cs Outdated Show resolved Hide resolved
backend/LocalWebApp/BackgroundSyncService.cs Show resolved Hide resolved
backend/MiniLcm/ILexboxApi.cs Outdated Show resolved Hide resolved
backend/MiniLcm/MultiString.cs Show resolved Hide resolved
@myieye
Copy link
Contributor

myieye commented May 24, 2024

1deb7d4 seems to indicate that the new abstract change classes were a good move. A bunch of code just vanished 🙂.

@hahn-kev hahn-kev merged commit 05f928d into develop May 28, 2024
13 of 14 checks passed
@myieye myieye deleted the feat/local-web-app branch May 28, 2024 15:15
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.

2 participants