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

Refactor error handling in File Browser plugin #1842

Open
mlewand opened this issue Mar 26, 2018 · 0 comments
Open

Refactor error handling in File Browser plugin #1842

mlewand opened this issue Mar 26, 2018 · 0 comments
Labels
changelog:skip A changelog entry should not be added for a given issue. plugin:filebrowser The plugin which probably causes the issue. plugin:filetools The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. type:task Any other issue (refactoring, typo fix, etc).

Comments

@mlewand
Copy link
Contributor

mlewand commented Mar 26, 2018 •
edited
Loading

Are you reporting a feature request or a bug?

Task

Provide detailed reproduction steps (if any)

In #1840 we made a quick fix in order to fix issue in found in #1840 (review).

It once again parses server response. Ideally the server response should be processed only once and shared as the processing continues.
@mlewand mlewand added type:task Any other issue (refactoring, typo fix, etc). target:minor Any docs related issue that can be merged into a master or major branch. changelog:skip A changelog entry should not be added for a given issue. plugin:filebrowser The plugin which probably causes the issue. plugin:filetools The plugin which probably causes the issue. labels Mar 26, 2018
@msamsel msamsel added the status:confirmed An issue confirmed by the development team. label Mar 27, 2018
@f1ames f1ames removed the target:minor Any docs related issue that can be merged into a master or major branch. label Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
changelog:skip A changelog entry should not be added for a given issue. plugin:filebrowser The plugin which probably causes the issue. plugin:filetools The plugin which probably causes the issue. status:confirmed An issue confirmed by the development team. type:task Any other issue (refactoring, typo fix, etc).
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@f1ames @mlewand @msamsel