Bulk fetch sms parts+cost - platform admin service list page - #2675
Open
whabanks wants to merge 11 commits into
Open
Bulk fetch sms parts+cost - platform admin service list page#2675whabanks wants to merge 11 commits into
whabanks wants to merge 11 commits into
Conversation
🧪 Review environmenthttps://cs6x7xq2entvytq2ma3scad2r40mukuf.lambda-url.ca-central-1.on.aws/ |
whabanks
marked this pull request as ready for review
April 27, 2026 14:47
jzbahrai
reviewed
Apr 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary | Résumé
This PR hooks up the platform admin services list page to the new bulk sms parts + cost API endpoint. Previously we fetched these stats one service at a time by id, which resulted in massive slow downs on the platform admin service list page load times and unnecessary load on the DB.
Page load performance comparison
Staging
Compares staging as is without the fragment + sms cost counts in the table to this PR running locally hooked up to staging.
1 month of data
Staging: 2s
This PR: 2.1s
4 months of data
Staging: 2.65s
This PR: 4s
1 Year of data
Staging: 5.1s
This PR: 6s
Prod
1 month of data
Prod: 3.4s
This PR: 3.8s
4 months of data
Prod: 12.3s
This PR: 16s
6 months of data
Prod: 23s
This PR: 23.7s
1 Year of data
Prod: 504 Gateway timeout
This PR: 504 Gateway timeout
Test instructions | Instructions pour tester la modification
Example:
--- Data gen script ---
/platform-admin/live-services> expand theApply filterssection2025-01-05>2025-09-05SMS partsandSMS costcolumns are updatedTo clean up your DB, add the
GENERATOR MARKER_TSdate output from the generation script into themarkersarray in the clean up script then execute it.--- Clean up script ---