Skip to content

Handles realtime with 2 states for environments table

Filipa Lacerda requested to merge environments-realtime into master

What does this MR do?

  • Merges the old state with the new to enable a better UX with realtime. No flickering is visible now.
  • Moves the isLoading property to the store so we can show only one loader. Currently more than one would be visible.
  • Keeps the same data structure because:
  1. allows a much smaller code change
  2. We would need to set some watchers to deal with updates in an object tree structure
  3. Allows not to repeat the same logic across several components.

Why was this MR needed?

In order to fix deploy boards code - https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/2311 https://gitlab.com/gitlab-org/gitlab-ee/issues/2677 - we first need to update the data in the ce repo

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ee/issues/2677

Edited by Filipa Lacerda

Merge request reports