-
Notifications
You must be signed in to change notification settings - Fork 4
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
chore: More migration to new Angular control flow #1804
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1804 +/- ##
==========================================
+ Coverage 84.60% 84.69% +0.08%
==========================================
Files 203 203
Lines 6691 6691
Branches 742 457 -285
==========================================
+ Hits 5661 5667 +6
Misses 866 866
+ Partials 164 158 -6 ☔ View full report in Codecov by Sentry. |
Hey @zusorio, great that you're looking into the migration of the control flow. We decided some time ago in the team to do it iteratively and to add stories for each touched component. So make sure that all components moved to the new control flow have stories if they make sense. If you spot CSS that can be moved to Tailwind, that's also a good place to do it. But as far as I can see, you're already on the right track :) |
9a16d71
to
db612f4
Compare
A Storybook preview is available for commit f9ce15d. |
This comment has been minimized.
This comment has been minimized.
db612f4
to
9d196bf
Compare
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A bit thanks for the migration and the stories 🫶
I have some smaller comments, in some cases I'd also add additional stories for specific cases.
frontend/src/app/projects/create-project/create-project.component.html
Outdated
Show resolved
Hide resolved
...d/src/app/projects/project-detail/edit-project-metadata/edit-project-metadata.component.html
Outdated
Show resolved
Hide resolved
frontend/src/app/projects/project-detail/edit-project-metadata/edit-project-metadata.stories.ts
Outdated
Show resolved
Hide resolved
...src/app/projects/project-detail/project-users/project-audit-log/project-audit-log.stories.ts
Show resolved
Hide resolved
...src/app/projects/project-detail/project-users/project-audit-log/project-audit-log.stories.ts
Show resolved
Hide resolved
frontend/src/app/settings/core/pipelines-overview/pipelines-overview.component.html
Outdated
Show resolved
Hide resolved
frontend/src/app/settings/core/pipelines-overview/pipelines-overview.stories.ts
Outdated
Show resolved
Hide resolved
frontend/src/app/settings/core/pipelines-overview/pipelines-overview.component.html
Show resolved
Hide resolved
9d196bf
to
25c1147
Compare
This comment has been minimized.
This comment has been minimized.
25c1147
to
3048bb7
Compare
This comment has been minimized.
This comment has been minimized.
3048bb7
to
4160d76
Compare
This comment has been minimized.
This comment has been minimized.
4160d76
to
5e49b07
Compare
This comment has been minimized.
This comment has been minimized.
5e49b07
to
5f081b1
Compare
This comment has been minimized.
This comment has been minimized.
5f081b1
to
0910006
Compare
This comment has been minimized.
This comment has been minimized.
Quality Gate failedFailed conditions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a million, especially for all of those great new stories 💯
frontend/src/app/projects/create-project/create-project.stories.ts
Outdated
Show resolved
Hide resolved
This report was generated by comparing f9ce15d with 69159ef. ArtifactName:
|
item | count |
---|---|
pass | 318 |
change | 0 |
new | 22 |
delete | 0 |
No description provided.