Skip to content
This repository has been archived by the owner on Jul 15, 2022. It is now read-only.

feature/rating ratings feature flag added #1930

Merged
merged 2 commits into from
May 3, 2022
Merged

Conversation

cgrellard-ledger
Copy link
Contributor

@cgrellard-ledger cgrellard-ledger commented May 3, 2022

Ratings feature flag added

LIVE-1860

Context (issues, jira)

Description / Usage

Expectations

  • Test coverage: The changes of this PR are covered by test. Unit test were added with mocks when depending on a backend/device.
  • No impact: The changes of this PR have ZERO impact on the userland. Meaning, we can use these changes without modifying LLD/LLM at all. It will be a "noop" and the maintainers will be able to bump it without changing anything.

@cgrellard-ledger cgrellard-ledger requested a review from a team as a code owner May 3, 2022 13:47
@vercel
Copy link

vercel bot commented May 3, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
ledger-live-common ❌ Failed (Inspect) May 3, 2022 at 2:35PM (UTC)

@cgrellard-ledger cgrellard-ledger merged commit d65a585 into develop May 3, 2022
@cgrellard-ledger cgrellard-ledger deleted the feature/rating branch May 3, 2022 15:46
This was referenced May 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants