-
Notifications
You must be signed in to change notification settings - Fork 5
Revert "Merge pull request #368 from elifesciences/312-push-messages" #518
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's see if re-instating those package versions make the pipeline work?
package.json
Outdated
@@ -139,7 +139,7 @@ | |||
"joi-browser": "^13.4.0", | |||
"lint-staged": "^7.2.2", | |||
"prettier": "^1.8.2", | |||
"pubsweet": "^2.3.0", | |||
"pubsweet": "^2.2.8", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think we should be downgrading these - probably what is causing the failures in the pipeline.
server/dar-server/package.json
Outdated
@@ -12,6 +12,6 @@ | |||
"express": "^4.16.2" | |||
}, | |||
"peerDependencies": { | |||
"pubsweet-server": "^8.0.0" | |||
"pubsweet-server": "^7.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think we should be downgrading these - probably what is causing the failures in the pipeline.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works in pipeline!
Background
This reverts the Websocket subscription, as it doesn't work when the app is served over https. This is due to the subscription url being prefixed with
ws://
in pubsweet-clientAny relevant tickets
#368, #507
How has this been tested?
CI