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

feat(contentful-apps): Stepper editor and Step editor apps #16435

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

RunarVestmann
Copy link
Member

@RunarVestmann RunarVestmann commented Oct 16, 2024

Stepper editor and Step editor apps

What

Why

Specify why you need to achieve this

Screenshots / Gifs

Attach Screenshots / Gifs to help reviewers understand the scope of the pull request

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Formatting passes locally with my changes
  • I have rebased against main before asking for a review

Copy link

codecov bot commented Oct 16, 2024

Codecov Report

Attention: Patch coverage is 0% with 180 lines in your changes missing coverage. Please review.

Project coverage is 36.61%. Comparing base (8b46f97) to head (569d24e).
Report is 42 commits behind head on main.

Files with missing lines Patch % Lines
...ful-apps/components/editors/stepper/StepEditor.tsx 0.00% 88 Missing ⚠️
...pps/contentful-apps/components/EntryListSearch.tsx 0.00% 48 Missing ⚠️
...-apps/components/editors/stepper/StepperEditor.tsx 0.00% 43 Missing ⚠️
...pps/pages/fields/generic-tag-group-items-field.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #16435      +/-   ##
==========================================
- Coverage   36.72%   36.61%   -0.11%     
==========================================
  Files        6809     6821      +12     
  Lines      141179   142092     +913     
  Branches    40248    40544     +296     
==========================================
+ Hits        51846    52025     +179     
- Misses      89333    90067     +734     
Flag Coverage Δ
contentful-apps 5.19% <0.00%> (-0.38%) ⬇️
web 1.81% <ø> (-0.01%) ⬇️

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

Files with missing lines Coverage Δ
...tors/lists/GenericListEditor/GenericListEditor.tsx 0.00% <ø> (ø)
apps/web/components/Stepper/utils.ts 0.00% <ø> (ø)
...pps/pages/fields/generic-tag-group-items-field.tsx 0.00% <0.00%> (ø)
...-apps/components/editors/stepper/StepperEditor.tsx 0.00% <0.00%> (ø)
...pps/contentful-apps/components/EntryListSearch.tsx 0.00% <0.00%> (ø)
...ful-apps/components/editors/stepper/StepEditor.tsx 0.00% <0.00%> (ø)

... and 59 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b46f97...569d24e. Read the comment docs.

@datadog-island-is
Copy link

datadog-island-is bot commented Oct 16, 2024

Datadog Report

Branch report: feature/contentful-apps-stepper-json-visuals
Commit report: 7587dd2
Test service: web

✅ 0 Failed, 84 Passed, 0 Skipped, 30.54s Total Time
➡️ Test Sessions change in coverage: 1 no change

@RunarVestmann RunarVestmann changed the title feat(contentful-apps): Stepper editor app and Step editor apps feat(contentful-apps): Stepper editor and Step editor apps Oct 21, 2024
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.

1 participant