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

feat(vscode): Improve Logic Apps sites API call #3338

Merged

ccastrotrejo merged 2 commits into main from ccastrotrejo/extensionImprovements

Sep 18, 2023

Contributor

ccastrotrejo commented Sep 18, 2023 •

edited

Loading

Main Code Chages

Update create client logic to fetch Logic Apps. This just make one single API call to fetch all of them instead of fetching 1 by one. This reduces the time it takes to load the LAs in the Resource tab and when trying to deploy.
Test Subscription was BTS4 which has around 750 Logic Apps. Last implementation would take ~ 30 minutes or more. New implementation takes ~1 minute.

The commit message follows our guidelines
Tests for the changes have been added (for bug fixes/features)
Docs have been added / updated (for bug fixes / features)

Fixes #2505

ccastrotrejo added 2 commits

September 15, 2023 15:55


          Improve logic apps loading in tree

d9ac628


          Merge branch 'main' of https://github.com/Azure/LogicAppsUX into ccas…

f317a62

…trotrejo/extensionImprovements

rllyy97 approved these changes

View reviewed changes

ccastrotrejo enabled auto-merge (squash)

September 18, 2023 20:17

ccastrotrejo merged commit abe421a into main

ccastrotrejo deleted the ccastrotrejo/extensionImprovements branch

September 18, 2023 20:43

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet