-
Notifications
You must be signed in to change notification settings - Fork 10
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
⬆️ bump open-api-framework to 0.8.0 #438
Conversation
034a2bd
to
25e61a1
Compare
25e61a1
to
daa3ad4
Compare
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.
@annashamray will you also add the warning about HSTS_SECONDS to the changelog? maykinmedia/open-api-framework#63 (comment)
- OBJECTS_OBJECTTYPES_CONFIG_ENABLE=false | ||
# - OBJECTTYPES_API_ROOT=https://objecttypes.example.com/api/v2/ |
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.
Is it intentional to disable 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.
Yes it's a temporary fix for quick-start action
OBJECTTYPES_API_ROOT
setting was incorrect (it's fixed in this PR), but quick-start doesn't use the current build, but only the published one.
So to make it work for now OBJECTTYPES_API_ROOT
should be empty
@stevenbal I planed to add the warning when preparing the release. Do you want me to add it now? |
@annashamray no that's fine, was just to double check so we don't forget about it 👍 |
After the PR is merged it would be added to patch release