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

client: asynchronous upload #21

Closed
wants to merge 17 commits into from
Closed

client: asynchronous upload #21

wants to merge 17 commits into from

Conversation

maybechris
Copy link
Contributor

No description provided.

@maybechris
Copy link
Contributor Author

maybechris commented Feb 3, 2023

@zhaofengli Apart from a few fixes/improvements the rust code should be final.1 I might however add nix modules and documentation.

Footnotes

  1. Although the cli isn't complete yet. Since this use case is quite similar to watch-store I was wondering if I should just alias Daemon to WatchStore (pub type Daemon = WatchStore;) and change the WatchStore fields to pub.

@maybechris maybechris mentioned this pull request Feb 22, 2023
@maybechris maybechris marked this pull request as ready for review March 5, 2023 17:59
@maybechris
Copy link
Contributor Author

@zhaofengli did you get a chance to look at the PR yet?

@zhaofengli
Copy link
Owner

Sorry, getting distracted by so much stuff. I'll hopefully get to this soon.

@maybechris
Copy link
Contributor Author

No worries, take all the time you need. I was just worrying that you might not have received a notification that the PR is ready to be reviewed.

@hmajid2301
Copy link

Hi is there any update on this ?

@maybechris maybechris closed this by deleting the head repository Dec 14, 2023
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.

None yet

3 participants