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

refactor: clean up UI #3288

Merged

Conversation

erka
Copy link
Collaborator

@erka erka commented Jul 20, 2024

No description provided.

@erka erka requested a review from a team as a code owner July 20, 2024 20:29
Copy link

codecov bot commented Jul 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.10%. Comparing base (2a0e3f4) to head (14627ca).

Additional details and impacted files
@@                  Coverage Diff                  @@
##           default-variant-redo    #3288   +/-   ##
=====================================================
  Coverage                 64.09%   64.10%           
=====================================================
  Files                       168      168           
  Lines                     13462    13462           
=====================================================
+ Hits                       8629     8630    +1     
+ Misses                     4164     4163    -1     
  Partials                    669      669           
Flag Coverage Δ
unittests 64.10% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@erka erka force-pushed the default-variant-redo-ui branch 2 times, most recently from f61475e to 90da495 Compare July 21, 2024 11:04
Signed-off-by: Roman Dmytrenko <[email protected]>
@erka erka force-pushed the default-variant-redo-ui branch from 90da495 to 14627ca Compare July 21, 2024 11:34
@markphelps markphelps merged commit 7cabab9 into flipt-io:default-variant-redo Jul 21, 2024
32 checks passed
@erka erka deleted the default-variant-redo-ui branch July 21, 2024 17:08
kodiakhq bot pushed a commit that referenced this pull request Jul 23, 2024
* feat: default variant redo

Signed-off-by: Mark Phelps <[email protected]>

* feat: add default variant in ui

Signed-off-by: Mark Phelps <[email protected]>

* chore: import/export

Signed-off-by: Mark Phelps <[email protected]>

* chore: ui lint issues

Signed-off-by: Mark Phelps <[email protected]>

* chore: add migrations for other dbs

Signed-off-by: Mark Phelps <[email protected]>

* chore: fix schemas; fix cli export test

Signed-off-by: Mark Phelps <[email protected]>

* chore: import export IT

Signed-off-by: Mark Phelps <[email protected]>

* chore: set IT log levels to warn to minimize log output

Signed-off-by: Mark Phelps <[email protected]>

* chore: rm dag import

Signed-off-by: Mark Phelps <[email protected]>

* chore: fix import/export IT

Signed-off-by: Mark Phelps <[email protected]>

* chore: fix ui test

Signed-off-by: Mark Phelps <[email protected]>

* chore: update testdata

Signed-off-by: Mark Phelps <[email protected]>

* chore: support evaluation data api

Signed-off-by: Mark Phelps <[email protected]>

* chore: make default variant optional

Signed-off-by: Mark Phelps <[email protected]>

* chore: rename default variant field in ui

Signed-off-by: Mark Phelps <[email protected]>

* refactor: clean up UI (#3288)

Signed-off-by: Roman Dmytrenko <[email protected]>

* chore: move variant form helper

Signed-off-by: Mark Phelps <[email protected]>

* chore: rename context key

Signed-off-by: Mark Phelps <[email protected]>

* chore: add legacy evaluator tests

Signed-off-by: Mark Phelps <[email protected]>

* chore: rm unused IVariant import

Signed-off-by: Mark Phelps <[email protected]>

* chore: db tests

Signed-off-by: Mark Phelps <[email protected]>

* chore: add IT

Signed-off-by: Mark Phelps <[email protected]>

* chore: check for default variant in same namespace

Signed-off-by: Mark Phelps <[email protected]>

* chore: add snapshot support

Signed-off-by: Mark Phelps <[email protected]>

* chore: try to fix ui ITs

* chore: with workdir

Signed-off-by: Mark Phelps <[email protected]>

* chore: go mod tidy

* chore: bump playwright/test

Signed-off-by: Mark Phelps <[email protected]>

---------

Signed-off-by: Mark Phelps <[email protected]>
Signed-off-by: Roman Dmytrenko <[email protected]>
Co-authored-by: Roman Dmytrenko <[email protected]>
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.

2 participants