-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
feat(sam): add sam support #1246
base: master
Are you sure you want to change the base?
Conversation
4105cee
to
0e692f0
Compare
Signed-off-by: GitHub <[email protected]>
The CI issues are related to below Line 88 in 8123f00
I think it should be removed since we have been created separate env for every single extra backend. We do not need a requirements.txt |
Signed-off-by: GitHub <[email protected]>
Signed-off-by: GitHub <[email protected]>
yup correct. consider for SAM we might need an additional endpoint in the API - maybe we can look around other closed API services to mimic their API? |
Description
This PR fixes #915
Notes for Reviewers
Please kindly check the this page to get more detail of SAM. It is a demo for our SAM support plan.
Further request
I do not know how to debug(one-step-by-one-step), please show me any useful information to help me test it.
Signed commits