-
Notifications
You must be signed in to change notification settings - Fork 1
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
api: added defaults #551
api: added defaults #551
Conversation
022fc72
to
6bbb8d0
Compare
transform: true, | ||
}), | ||
); | ||
// TODO: should get replaced with the config |
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.
seems like config in graph api is not refactored yet!
@@ -23,7 +23,7 @@ x-common-environment: &common-environment | |||
|
|||
x-content-publishing-env: &content-publishing-env | |||
START_PROCESS: content-publishing-api | |||
FILE_UPLOAD_MAX_SIZE_IN_BYTES: 2000000000 | |||
FILE_UPLOAD_MAX_SIZE_IN_BYTES: 500000000 |
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.
it seems like we can only upload files up to 500MB based on current setup and redis
@@ -334,89 +334,6 @@ | |||
"health" | |||
] | |||
} | |||
}, |
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.
for some reason my IDE likes a lot to apply this change.
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.
- Read through changes, everything makes sense to me, nice work.
- One non-blocking typo
- Did not test
🚢 it!
Co-authored-by: Matthew Orris <[email protected]>
Problem
Related to #491
Solution
with @pairperson1
Steps to Verify: