Skip to content
Snippets Groups Projects

feat: add map full data mgmt

Context:

In !35 (merged) we've prepared base of map data management but without covering aspect of communication with API. !35 (merged) creates only solution for handling client data.

Objective:

This PR adds data management of map data fetched from the backend.

It includes:

  • redux boilerplate & fetching of: map, backgrounds, overlays, models
  • fix of redux boilerplate: project (previously the default value was an empty array)
  • unit tests of the above

What is not included:

  • whole business logic of firing actions and using this data, it'll be excluded to other PR

After draft

Additionally:

  • extracted reducers to separate variable as it's completely unnecessary to copy this object into utils every time we add new reducer
Edited by Adrian Orłów

Merge request reports

Merge request pipeline #80302 passed

Merge request pipeline passed for 8eea9f3d

Test coverage 86.35% (-1.49%) from 1 job

Merged by Adrian OrłówAdrian Orłów 1 year ago (Oct 24, 2023 12:48pm UTC)

Loading

Pipeline #80323 passed

Pipeline passed for 77c58674 on development

Test coverage 86.35% (-1.49%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tadeusz Miesiąc
  • Tadeusz Miesiąc
  • Tadeusz Miesiąc
  • Models store and query is already in production. It's RFC for me

  • Adrian Orłów added 41 commits

    added 41 commits

    Compare with previous version

  • Adrian Orłów changed the description

    changed the description

  • Adrian Orłów added 3 commits

    added 3 commits

    • 52e67bf5 - fix: models non consistent
    • ec99154f - fix: make zod seed an env
    • 8eea9f3d - test: add tests for backgrounds, overlays, project

    Compare with previous version

  • Adrian Orłów marked this merge request as ready

    marked this merge request as ready

  • Tadeusz Miesiąc approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading