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

Feature/mowdev 3411 new #237

Closed
wants to merge 68 commits into from
Closed

Feature/mowdev 3411 new #237

wants to merge 68 commits into from

Conversation

shashi165
Copy link
Contributor

No description provided.

shashi165 and others added 30 commits December 4, 2019 23:06
mowdev-3411
3b04304
added more rules
b36efd3
@lewisdaly
Feature/846 async logging (#100)
6567058 
* Add file logging support in docker, update dependencies

* Add back missing package after merge conflicts
@partiallyordered
Feature/update json rules engine (#101)
03fc69d 
* Simplified rules unit tests

* Updated json-rules-engine

* Updated code to use native json-rules-engine jsonpath syntax

* Made linter happy. Removed jsonpath dependency.
@partiallyordered
Feature/test fx rules (#102)
78bdc53 
* Added skeleton for fx rules unit tests

* Removed commented tests from original file

* Update fx.test.js
added more rules and tests
71dff00
test discard
7a5cb58
test discard
6b058cc
test discard
1dd36cb
stripe off accept header for PUT requests
28549b8
added package-lock.json
621ff7c
@shashi165
Merge branch 'master' into feature/mowdev-3411
61c4dd0
added more rules
e27d8dc
Merge remote-tracking branch 'origin/feature/mowdev-3411' into featur…
72e1dac 
…e/mowdev-3411
fixed the package.json version
da91411
fixed version number
b6b3890
fixed the Object.assign
4e4db82
removed rules.json
578f61a
update package-lock.json
026bbf5
@shashi165
Merge branch 'master' into feature/mowdev-3411
200d721
@KamuelaFranco
Attempt cache refresh
3a87289
@KamuelaFranco
Replace audit:check with audit
cbb418f
@KamuelaFranco
Debug with verbose audit
2c5d6f6
@KamuelaFranco
Skip vulnerability check because of network errors
1be3256
@KamuelaFranco
Skip vuln check step
736f5b3
downgrade helm version
693f3ae
temp logging
58a285b
renamed switchEndpoint to a better name
99ceeb5
@shashi165
removed config from dockerfile
745d45f
@partiallyordered
Added synchronous responses for rules engine invalid quote errors (#127)
f6e8f6c 
* Added synchronous responses for rules engine invalid quote errors

* Corrected import

* Corrected error variable name

* Moved response handling out of model, into handlers

* Fixed tests
Vassilis Barzokas and others added 22 commits March 18, 2020 14:06
Do not modify headers in case they are only being relayed to another …
50858fe 
…DFSP.
Do not delete FSPIOP-Signature header unless if modifyHeaders is …
76e654a 
…`true`.
Merged master into this branch.
c9cde80
merge master
de7996b
fixed the bug with createQuoteExtensions
464b4da
fixed issue with createQuoteExtension
5a49911
Revert change of the error message as it breaks tests.
8c735ae
Aligned an error message with the master branch to match Postman test…
05b7dfc 
…'s assertions.
Fixed some npm vulnerabilities and skipped the rest.
39121ef
Updated circle CI config according to master branch.
1261ad1
Removed quoteId from error log message as it causes error due its e…
2669bdd 
…xceeding length.
fixed error message
4303335
fix error message
aef86fa
fix error message
88e85be
merge master
93f0420
merge branch mowdev-3411
ff1c180
linting changes
0ebf068
linting changes
08dbb68
linting changes
9336dcd
linting changes
5faf73b
merging master
dd33bf8
merge master
d74ea2d
This was referenced Aug 25, 2020
Open
Open
@CLAassistant
Copy link

CLAassistant commented Oct 26, 2020 •
edited
Loading

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 5 committers have signed the CLA.

✅ shashi165
✅ lewisdaly
✅ partiallyordered
❌ vbarzokas
❌ KamuelaFranco

shashi165 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@elnyry-sam-k elnyry-sam-k deleted the feature/mowdev-3411-new branch February 9, 2021 07:03
@elnyry-sam-k elnyry-sam-k restored the feature/mowdev-3411-new branch June 2, 2021 15:21
@snyk-bot snyk-bot mentioned this pull request Jan 12, 2022
Open
@snyk-bot snyk-bot mentioned this pull request Feb 9, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@shashi165 @CLAassistant @elnyry-sam-k @lewisdaly @partiallyordered @KamuelaFranco