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

integrate zones list endpoint MAASENG-1975 #5071

Merged

Conversation

petermakowski
Copy link
Contributor

@petermakowski petermakowski commented Jul 20, 2023

Done

  • integrate zones list endpoint MAASENG-1975
    • call /MAAS/a/v1/zones REST endpoint instead of zone.list websocket API whenever zone/fetch redux action is called
    • reduce redundant tests nesting in describe("http sagas", () => {

QA

QA steps

  • open developer tools and go to Network tab
  • Go to machines list
  • Verify that zones list has been fetched via REST API (should be in Fetch/XHR list) and there is no zone.list in websocket messages

Fixes

Fixes: https://warthogs.atlassian.net/browse/MAASENG-1975

@webteam-app
Copy link

Demo starting at https://maas-ui-5071.demos.haus

@petermakowski petermakowski marked this pull request as ready for review July 20, 2023 09:37
@petermakowski petermakowski force-pushed the zones-list-endpoint-MAASENG-1975 branch 2 times, most recently from 6228aef to 6566331 Compare July 20, 2023 09:42
@petermakowski petermakowski merged commit cb08715 into canonical:main Jul 20, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants