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.

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 flow changes #865

Draft
wants to merge 19 commits into
base: master
Choose a base branch
from
Draft

API flow changes #865

wants to merge 19 commits into from

Conversation

azazeal
Copy link
Contributor

@azazeal azazeal commented Mar 22, 2022

This PR reduces boilerplate and addresses error handling in the api package and its sub-packages..

@github-actions github-actions bot added the needs triage Waiting for discussion / prioritization by team label Mar 22, 2022
@azazeal azazeal force-pushed the panos/api/render branch 4 times, most recently from b2fa8e3 to 3d31449 Compare March 29, 2022 13:35
Base automatically changed from panos/api/render to master March 30, 2022 08:22
@hslatman
Copy link
Member

hslatman commented May 4, 2023

The changes in this PR seem to have been merged already for the most part with #860. This one can probably be closed?

@azazeal
Copy link
Contributor Author

azazeal commented May 4, 2023

The changes in this PR seem to have been merged already for the most part with #860. This one can probably be closed?

Looks like it but I'll rebase the beast just to make sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Waiting for discussion / prioritization by team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants