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

Dev infra #8241

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Dev infra #8241

wants to merge 5 commits into from

Conversation

ganthern
Copy link
Contributor

@ganthern ganthern commented Jan 9, 2025

No description provided.

@ganthern ganthern marked this pull request as draft January 10, 2025 09:03
@tuta-sudipg tuta-sudipg force-pushed the dev-infra branch 3 times, most recently from ef41bb6 to d0edb8b Compare January 16, 2025 14:59
@ganthern ganthern force-pushed the dev-infra branch 2 times, most recently from a2e991d to 6eb3d92 Compare January 16, 2025 16:12
we're moving to persisting the import state only on the server,
with the files to be imported on disk the only local state that
is kept by the importer.

Most errors happen during import preparation after we receive the files
to import. These can be caught & handled immediately. If no errors
happen, the remote state will be set up on the server at this stage and
can be subscribed to by the UI by loading it via its ID

Some other errors can happen asynchronously while the importer is
running in the background. For those, the importer sets a hook to
receive and handle them as they happen.

any normal state updates are sent to the server in node-mimimi and
received in the UI via entity updates.

Co-authored-by: nig <[email protected]>
Co-authored-by: sug <[email protected]>
Co-authored-by: map <[email protected]>
Co-authored-by: jhm <[email protected]>
Co-authored-by: Kinan <[email protected]>
Co-authored-by: abp <[email protected]>
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.

3 participants