FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(file-system): async read/write by PeterStaev · Pull Request #7671 · NativeScript/NativeScript · GitHub

/ NativeScript Public

feat(file-system): async read/write - #7671

Merged
manoldonev merged 13 commits into
NativeScript:masterfrom
PeterStaev:issue-2239
Oct 2, 2019
ConversationCommits (13)ChecksFiles changed
Merged

feat(file-system): async read/write#7671
manoldonev merged 13 commits into
NativeScript:masterfrom
PeterStaev:issue-2239

Conversation

PeterStaev commented Aug 11, 2019 •
edited
Loading

Copy link
Copy Markdown
Contributor

PR Checklist

What is the current behavior?

Current file-system methods for read/write files are synchronous which makes the UI thread block if a big file is read/written.

What is the new behavior?

True async methods have been added that use native code inside tns-core-modules-widgets execute the read/write operations in a separate thread.

Closes #2239 .

cla-bot Bot added the cla: yes label Aug 11, 2019
PeterStaev changed the title feat(files-system): async read/write feat(file-system): async read/write Aug 11, 2019

This comment was marked as abuse.

Sign in to view

PeterStaev commented Aug 12, 2019

Copy link
Copy Markdown
Contributor Author

Yes, this is a possible solution, if breaking changes in minor versions are considered that bad :)

etabakov commented Aug 13, 2019

Copy link
Copy Markdown
Contributor

Hey, @PeterStaev. Regarding the breaking changes policy, our team is committed to following the SemVer Specification, which means that all changes in a minor version should be introduced in a backward-compatible manner. With that said, Nathanael's suggestion is a good one. Would you mind adjusting your PR according to his proposal, so that we can proceed with the technical review?

Thanks for jumping on this issue!

PeterStaev commented Aug 13, 2019

Copy link
Copy Markdown
Contributor Author

@etabakov , updated and removed the breaking changes. Removed from the above description as well :)

manoldonev self-requested a review August 21, 2019 13:53

isabelh340 commented Sep 10, 2019

Copy link
Copy Markdown

Alot

manoldonev reviewed Sep 18, 2019
View reviewed changes
Comment thread tns-core-modules/file-system/file-system.ts Outdated
manoldonev reviewed Sep 18, 2019
View reviewed changes
Comment thread tns-core-modules/file-system/file-system.ts Outdated

manoldonev commented Oct 1, 2019

Copy link
Copy Markdown
Contributor

test

1 similar comment

SvetoslavTsenov commented Oct 1, 2019

Copy link
Copy Markdown
Contributor

test

manoldonev added the docs needed Additional documentation on this issue/PR is needed label Oct 2, 2019
manoldonev merged commit 2146ac9 into NativeScript:master Oct 2, 2019

manoldonev commented Oct 2, 2019

Copy link
Copy Markdown
Contributor

@PeterStaev Looking good now and I have merged the PR. Thanks!
Will check what exactly is the process with https://www.bountysource.com/issues/34832190-make-the-file-system-module-run-in-a-background-thread and someone with the NativeScript team will follow up with you.

manoldonev mentioned this pull request Oct 2, 2019
Closed
PeterStaev deleted the issue-2239 branch October 2, 2019 15:09
vakrilov mentioned this pull request Oct 25, 2019
Merged
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
manoldonev manoldonev approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yes ♥ community PR docs needed Additional documentation on this issue/PR is needed

Projects

NativeScript Docs Project
Status: 🆕 New

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Make the file-system module run in a background thread [$300]

7 participants

Back | FazBrowse Home | New Git URL