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(quick-start): Add logic to skeleton - Part 2 #79179

Merged

Conversation

priscilawebdev
Copy link
Member

@priscilawebdev priscilawebdev commented Oct 16, 2024

This PR adds logic for 'waiting' on events/errors and implements the task-skipping functionality.

Contributes to https://github.com/getsentry/projects/issues/259

Preview

Screen.Recording.2024-10-16.at.09.06.15.mov

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Oct 16, 2024
Copy link

codecov bot commented Oct 16, 2024

Codecov Report

Attention: Patch coverage is 3.33333% with 29 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...tic/app/components/onboardingWizard/newSidebar.tsx 0.00% 15 Missing ⚠️
...tic/app/components/onboardingWizard/taskConfig.tsx 6.66% 14 Missing ⚠️
Additional details and impacted files
@@                                      Coverage Diff                                       @@
##           priscila/feat/quick-start/add-new-design-skeleton-feedback   #79179      +/-   ##
==============================================================================================
- Coverage                                                       78.26%   78.26%   -0.01%     
==============================================================================================
  Files                                                            7136     7136              
  Lines                                                          314133   314149      +16     
  Branches                                                        51293    51295       +2     
==============================================================================================
- Hits                                                           245871   245870       -1     
- Misses                                                          61836    61853      +17     
  Partials                                                         6426     6426              

…ack' into priscila/feat/quick-start/add-logic-to-skeleton-part-2
Base automatically changed from priscila/feat/quick-start/add-new-design-skeleton-feedback to master October 16, 2024 12:23
) => {
if (p.hasQuickStartUpdatesFeature) {
return (
<div {...p}>
Copy link
Member

Choose a reason for hiding this comment

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

You are spreading text & hasQuickStartUpdatesFeature onto the div . We should omit them.

@priscilawebdev priscilawebdev deleted the priscila/feat/quick-start/add-logic-to-skeleton-part-2 branch October 17, 2024 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants