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

Audit Template Report Submission Endpoint #4411

Merged
merged 8 commits into from
Nov 23, 2023
Merged

Conversation

kflemin
Copy link
Contributor

@kflemin kflemin commented Nov 17, 2023

New endpoint to retrieve an Audit Template report submission (either in PDF or XML format)
For 179D.

Copy link

Label error. Requires at least 1 of: Feature, Bug, Enhancement, Maintenance, Documentation, Performance, Do not publish. Found:

@nllong nllong added the Feature Add this label to new features. This will be reflected in the change log when generated. label Nov 18, 2023
Copy link
Member

@nllong nllong left a comment

Choose a reason for hiding this comment

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

let me test this one... give me a couple days.

Copy link
Member

@nllong nllong left a comment

Choose a reason for hiding this comment

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

sweet, worked as expected through swagger!

BSync even returned!
image

@@ -161,7 +196,7 @@ def build_xml(self, state, report_type, display_field):
view = state.propertyview_set.first()

gfa = state.gross_floor_area
if type(gfa) == int:
if isinstance(gfa, int):
Copy link
Member

Choose a reason for hiding this comment

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

silly python thing...

@@ -133,7 +133,7 @@ def health_check(request):
celery_status = False

try:
redis_status = not cache.has_key('redis-ping')
redis_status = 'redis-ping' not in cache
Copy link
Member

Choose a reason for hiding this comment

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

autopep caught this...

@@ -17,13 +17,57 @@


class AuditTemplateViewSet(viewsets.ViewSet, OrgMixin):
@swagger_auto_schema(manual_parameters=[
AutoSchemaHelper.query_org_id_field(),
AutoSchemaHelper.base_field(
Copy link
Member

Choose a reason for hiding this comment

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

my few updates to have swagger expose the correct info.

@nllong nllong merged commit bfe6765 into develop Nov 23, 2023
7 checks passed
@nllong nllong deleted the 179d_at_submissions branch November 23, 2023 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Add this label to new features. This will be reflected in the change log when generated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants