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

feat(api): set api variable to expose by default #768

Merged
merged 5 commits into from
Jul 12, 2024
Merged

feat(api): set api variable to expose by default #768

merged 5 commits into from
Jul 12, 2024

Conversation

gphorvath
Copy link
Contributor

No description provided.

@gphorvath gphorvath added the tech-debt Not a feature, but still necessary label Jul 12, 2024
@gphorvath gphorvath added this to the EVERGREEN milestone Jul 12, 2024
@gphorvath gphorvath self-assigned this Jul 12, 2024
@gphorvath gphorvath requested a review from a team as a code owner July 12, 2024 13:53
@gphorvath gphorvath linked an issue Jul 12, 2024 that may be closed by this pull request
Exposing API variable naming - confusing #734
Closed
@netlify Netlify
Copy link

netlify bot commented Jul 12, 2024 •
edited
Loading

✅ Deploy Preview for leapfrogai-docs canceled.

Name Link
🔨 Latest commit 61f15f6
🔍 Latest deploy log https://app.netlify.com/sites/leapfrogai-docs/deploys/669159a8c1f70000084b25c2

YrrepNoj
YrrepNoj reviewed Jul 12, 2024
View reviewed changes
uds-bundles/dev/README.md Outdated Show resolved Hide resolved
@gphorvath @YrrepNoj
Update uds-bundles/dev/README.md
037b265 
Co-authored-by: Jonathan Perry <YrrepNoj@gmail.com>
YrrepNoj
YrrepNoj previously approved these changes Jul 12, 2024
View reviewed changes
Copy link
Member

@YrrepNoj YrrepNoj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gphorvath gphorvath changed the title chore(api): change expose_openapi_schema variable name feat(api): set api variable to expose by default Jul 12, 2024
@gphorvath gphorvath enabled auto-merge (squash) July 12, 2024 17:21
@gphorvath gphorvath merged commit 310a735 into main Jul 12, 2024
14 checks passed
@gphorvath gphorvath deleted the 734-exposing-api-variable-naming-confusing branch July 12, 2024 17:24
@github-actions github-actions bot mentioned this pull request Jul 12, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YrrepNoj YrrepNoj YrrepNoj approved these changes

Assignees

@gphorvath gphorvath

Labels
tech-debt Not a feature, but still necessary
Projects
None yet
Milestone
EVERGREEN
Development

Successfully merging this pull request may close these issues.

Exposing API variable naming - confusing
2 participants
@gphorvath @YrrepNoj