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

Full Fire Marshal Report #564

Merged
merged 2 commits into from
Jun 12, 2023
Merged

Full Fire Marshal Report #564

merged 2 commits into from
Jun 12, 2023

Conversation

johnyu95
Copy link
Contributor

@johnyu95 johnyu95 commented Jun 7, 2023 •
edited
Loading

This PR contains the following:

  • Adds Full Fire Marshal Report to FDNY custom forms.
  • Text updates to the Fire Marshal Face Sheet custom form.
  • Adds support for ordering custom forms based on the new order column.
  • Updates Open Data Compliance Report to filter on Response date instead of Request date.

@johnyu95 johnyu95 requested a review from zgary June 7, 2023 19:56
zgary
zgary approved these changes Jun 9, 2023
View reviewed changes
Copy link
Contributor

@zgary zgary left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@johnyu95 johnyu95 merged commit ec9f38c into main Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zgary zgary zgary approved these changes

Assignees

@zgary zgary

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@johnyu95 @zgary