-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
configure CORS on server without api #4521
Conversation
8661138
to
7decd47
Compare
@wochinge Tested this locally -- before, it worked with the flag and didn't work without, and after, it worked with both. Let me know if you have ideas for a more proper way of testing this! |
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.
Thanks for fixing that! However, let's re-use the cors stuff between the different configure_app
s
Co-Authored-By: Tobias Wochinger <t.wochinger@rasa.com>
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 good to me!
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.
🚀
Co-Authored-By: Tobias Wochinger <t.wochinger@rasa.com>
Proposed changes:
--enable-api
#4463Status (please check what you already did):
black
(please check Readme for instructions)