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

CMDCT-2794 (Pt. II): Setup MCR w/ Zustand for State Management #11520

Closed
wants to merge 15 commits into from

Conversation

karla-vm
Copy link
Collaborator

Description

Follow-up PR for Part I.

This is a stacked PR covers the Admin Banner setup and its corresponding tests.

Related ticket(s)

CMDCT-2974


How to test

Sign in as an Admin and make sure you're able to create and delete banners under My Account

Important updates


Author checklist

  • I have performed a self-review of my code
  • I have added thorough tests, if necessary
  • I have updated relevant documentation, if necessary

convert to a different template: test → val | val → prod

Copy link

codeclimate bot commented Nov 14, 2023

Code Climate has analyzed commit c522b7f and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 1

The test coverage on the diff in this pull request is 77.9% (90% is the threshold).

This pull request will bring the total coverage in the repository to 95.7% (-0.2% change).

View more on Code Climate.

@gmrabian gmrabian changed the base branch from main to zustand-2974 November 15, 2023 15:00
Copy link
Contributor

@gmrabian gmrabian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thoughts, likely non-blocking

services/ui-src/src/components/pages/Home/HomePage.tsx Outdated Show resolved Hide resolved
services/ui-src/src/types/banners.ts Outdated Show resolved Hide resolved
karla-vm and others added 13 commits November 15, 2023 13:18
…rs-zus-2974"

This reverts commit 0b02e3a, reversing
changes made to ca5d93c.
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@karla-vm karla-vm closed this Nov 17, 2023
@karla-vm karla-vm deleted the banners-zus-2974 branch April 17, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants