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

Export form feature summary as CSV #2561

Merged
merged 41 commits into from
Feb 27, 2024
Merged

Conversation

StevenLeighton21
Copy link
Contributor

@StevenLeighton21 StevenLeighton21 commented Feb 21, 2024

Generate a form summary on request for a given environment, outputs as a csv

This can then be used to extrapolate a lot of interesting starts like how many forms use a given component or enable a given feature.

In dev, I generated a report on 29 services in 2.6 seconds

The major expensive operation is that each form needs to have its metadata pulled from the api as a single request for the latest version, in live envs this is likely to be 10x slower, but should still be able to generate a report in under 1 minute.

Report generation is only accessible as a logged in administrator via the admin UI

Copy link
Member

@zheileman zheileman left a comment

Choose a reason for hiding this comment

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

A few suggestions! looking great 🥳

@StevenLeighton21 StevenLeighton21 changed the title First WIP Export form feature summary as CSV Feb 26, 2024
@StevenLeighton21 StevenLeighton21 marked this pull request as ready for review February 26, 2024 16:31
@StevenLeighton21 StevenLeighton21 requested a review from a team as a code owner February 26, 2024 16:31
@@ -62,8 +62,52 @@ def export_services
end
end

def export_dev_form_summary
Copy link
Member

Choose a reason for hiding this comment

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

Not a biggie but these 2 methods feel like could be just one that receives a couple arguments? 😄

Copy link
Member

@zheileman zheileman left a comment

Choose a reason for hiding this comment

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

🚢

Copy link
Contributor

@H3ll3m4 H3ll3m4 left a comment

Choose a reason for hiding this comment

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

🚀

@StevenLeighton21 StevenLeighton21 merged commit 7a7be7a into main Feb 27, 2024
14 checks passed
@StevenLeighton21 StevenLeighton21 deleted the testable-forms-feature-report branch February 27, 2024 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants