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

Remote Loop overrides #5043

Merged
merged 25 commits into from
Oct 16, 2019
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
78661f5
skeleton code
ps2 Sep 26, 2019
7cfc7db
publishing
ps2 Sep 26, 2019
4348f0e
Activating remote override
ps2 Sep 26, 2019
21c7af1
Fix event types refresh, and fix custom duration
ps2 Sep 26, 2019
7d77a1c
Don't create treatment record; loop will
ps2 Sep 30, 2019
f189997
Additional override details for treatment tooltip
ps2 Sep 30, 2019
cbc4fd5
Pass client from careportal
ps2 Sep 30, 2019
447645d
attempt at moving endpoint to api/v2
ps2 Sep 30, 2019
600e7d4
Add bodyParser to v2/notifications
ps2 Sep 30, 2019
b31cf03
Remove notifications api from v1
ps2 Sep 30, 2019
6c33a91
Remove unused var
ps2 Sep 30, 2019
3e4895a
Better error handling and reporting
ps2 Oct 1, 2019
c9bb2fb
Add documentation for new loop plugin vars
ps2 Oct 1, 2019
b089d99
Handle indefinite duration overrides
ps2 Oct 1, 2019
5f234ee
Move label of indeterminate duration override to now line when start …
ps2 Oct 1, 2019
1e3de84
test
ps2 Oct 2, 2019
9078b24
Use separate representation for indefinite duration treatments
ps2 Oct 6, 2019
02a94c4
Merge remote-tracking branch 'origin/dev' into remote-loop-overrides
ps2 Oct 6, 2019
5004d15
fix display for finite duration treatments
ps2 Oct 6, 2019
77972cd
Merge remote-tracking branch 'origin/dev' into remote-loop-overrides
ps2 Oct 9, 2019
7a95706
Fix re-use of elements when treatment ordering changes
ps2 Oct 9, 2019
dbde581
Handle more rect transition() updates
ps2 Oct 9, 2019
bce2351
Add check for LOOP_DEVELOPER_TEAM_ID
ps2 Oct 13, 2019
0611455
Support configuration of aps-environment via environment variable.
ps2 Oct 14, 2019
63ac1dd
Fix expiration date math; was essentially preventing retries
ps2 Oct 16, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add bodyParser to v2/notifications
  • Loading branch information
ps2 committed Sep 30, 2019
commit 600e7d4901b3d47ccaebadd4dec5d300a40ad745
4 changes: 4 additions & 0 deletions app.js
Original file line number Diff line number Diff line change
Expand Up @@ -169,6 +169,10 @@ function create (env, ctx) {
limit: 1048576 * 50
}), api);

app.use('/api/v2', bodyParser({
limit: 1048576 * 50
}), api);

app.use('/api/v2/properties', ctx.properties);
app.use('/api/v2/authorization', ctx.authorization.endpoints);
app.use('/api/v2/ddata', ddata);
Expand Down
3 changes: 0 additions & 3 deletions lib/api/notifications-v2.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,6 @@ function configure (app, ctx) {
;

api.post('/loop', ctx.authorization.isPermitted('notifications:loop:push'), function (req, res) {
console.log("req = ", req);
console.log("req.connection.remoteAddress = ", req.connection.remoteAddress);
console.log("req.body = ", req.body);
ctx.loop.sendNotification(req.body, req.connection.remoteAddress, function (error) {
if (error) {
res.sendStatus(500);
Expand Down