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

SSE client #159

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

SSE client #159

wants to merge 6 commits into from

Conversation

adomasven
Copy link
Member

Adds SSE support and the first feature we're using it for - attaching current page as a snapshot or link to a selected item.

Closes #102

@adomasven
Copy link
Member Author

Requires a modification to reports endpoint to accept plaintext, because this update significantly improves the look of error and debug reports and also includes error/debug info from the client if it is running. The text of the report is generated on the client side.

See 000978c#diff-f4d3a666fd0dcb74381901a1aeb1044eR131

@adomasven
Copy link
Member Author

Also, the zotero/zotero PR needs to be merged first, and then this PR needs to update the submodule and a zotero src file reference in the gulpfile.

@adomasven
Copy link
Member Author

Updated

@adomasven adomasven force-pushed the feature/sse-client branch 2 times, most recently from 448be8c to 1e49143 Compare May 20, 2020 08:55
@adomasven
Copy link
Member Author

Closes #86

@adomasven
Copy link
Member Author

Closes #163

Allows proxy pref sharing across browsers on the same computer
and proxy retention when a browser profile is reset/erased

NOTE: Currently disabled due to required schema changes in the client
Also updates for recent versions of browsers to run with travis-ci again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Context-menu option to add current page as attachment
1 participant