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

Add example app for WASM / Web Sync #5880

Merged
merged 55 commits into from
Jun 21, 2023
Merged

Add example app for WASM / Web Sync #5880

merged 55 commits into from
Jun 21, 2023

Conversation

elle-j
Copy link
Contributor

@elle-j elle-j commented Jun 9, 2023 •
edited
Loading

What, How & Why?

Adds an example React Todo/Task app for showcasing Realm and Sync for Web using WASM.

For more information see the Example WASM README.

Use cases

  • Log in and register (email/password)
  • Log out
  • Create tasks
  • Read/query tasks
  • Update the status of tasks
  • Delete tasks
  • Sync
  • Local/Offline-first
  • Realm JS and @realm/react hooks

elle-j added 30 commits June 1, 2023 12:32
@elle-j
Set up and configure React app.
cca765b
@elle-j
Add 'Task' model.
db34fb1
@elle-j
Add 'TaskItem' component.
5d2b0d9
@elle-j
Add 'TaskList' component.
f9b98e0
@elle-j
Add 'AddTaskForm' component.
62a00d2
@elle-j
Add 'IntroText' component.
5658064
@elle-j
Add 'useTaskManager' hook.
a64902f
@elle-j
Add 'TaskScreen' screen/page.
484b6d6
@elle-j
Update 'App' component w/ 'RealmProvider' and 'TaskScreen'.
bbada76
@elle-j
Remove 'source-map-support'.
a87c47d
@elle-j
Add 'realm' and 'realm/react' to deps.
f87423c
@elle-j
Add example app to workspaces.
9aa2779
@elle-j
Update userId prop schema.
7eda710
@elle-j
Add 'useAppManager' hook.
d5b6cb9
@elle-j
Add sync and router.
73fba87
@elle-j
Add form to 'LoginScreen'.
8557991
@elle-j
Update Atlas App ID.
58e6054
@elle-j
Update router.
663f6a5
@elle-j
Refactor routing.
a61c66a
@elle-j
Add styles to login page.
736c521
@elle-j
Remove explicit React imports.
8616436
@elle-j
Add styles to tasks page.
a54f291
@elle-j
Fix rerendering issue.
5a23eb2
@elle-j
Minor updates.
01ed826
@elle-j
Memoize 'TaskItem'.
4761e78
@elle-j
Move assets to own directory.
bc744dd
@elle-j
Remove 'test' script.
81765a1
@elle-j
Remove memoization of 'TaskItem'.
3c76de9 
The 'isComplete' prop is incorrectly showing the same value for 'prevTask' and 'nextTask' when toggling 'isComplete'. The 'shouldNotRerender()' function therefore returns 'true' even when 'isComplete' is toggled via the UI.
@elle-j
Update README.
bde28a0
@elle-j
Update README.
8821c14
takameyer
takameyer reviewed Jun 13, 2023
View reviewed changes
example-wasm/README.md Outdated Show resolved Hide resolved
nhachicha
nhachicha approved these changes Jun 15, 2023
View reviewed changes
Copy link
Contributor

@nhachicha nhachicha left a comment

Choose a reason for hiding this comment

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

Nice one 🎉
I tested this using the latest nh/wasm/emscripten_target which leverages the Bindgen Opt-In, as well as building WASM in Release mode and it works 👍

takameyer
takameyer approved these changes Jun 16, 2023
View reviewed changes
Copy link
Contributor

@takameyer takameyer left a comment

Choose a reason for hiding this comment

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

Great work on this! I added a few minor comments. Also I noticed that there are linter errors unrelated to this code. They seem to preexist from emscripten_target. We should probably make a ticket to remove those, or help @nhachicha setup his linter :D

kraenhansen
kraenhansen reviewed Jun 21, 2023
View reviewed changes
package.json Outdated Show resolved Hide resolved
@elle-j elle-j merged commit b461896 into nh/wasm/emscripten_target Jun 21, 2023
@elle-j elle-j deleted the lj/wasm/poc branch June 21, 2023 10:23
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kneth kneth kneth left review comments

@kraenhansen kraenhansen kraenhansen left review comments

@takameyer takameyer takameyer approved these changes

@nhachicha nhachicha nhachicha approved these changes

@fealebenpae fealebenpae Awaiting requested review from fealebenpae

Assignees

@elle-j elle-j

Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@elle-j @takameyer @kneth @kraenhansen @nhachicha