-
Notifications
You must be signed in to change notification settings - Fork 79
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
Remove interaction_elements_ids from Component trait. #695
Remove interaction_elements_ids from Component trait. #695
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #695 +/- ##
==========================================
+ Coverage 90.13% 90.18% +0.04%
==========================================
Files 76 76
Lines 10059 10054 -5
Branches 10059 10054 -5
==========================================
Hits 9067 9067
+ Misses 910 905 -5
Partials 82 82 ☔ View full report in Codecov by Sentry. |
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.
Reviewed 7 of 7 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @alonh5)
1e85cf1
to
239af41
Compare
8d6d5dc
to
5213747
Compare
239af41
to
70c76e0
Compare
5213747
to
db449ac
Compare
70c76e0
to
640151c
Compare
db449ac
to
02de561
Compare
640151c
to
b9ac5d3
Compare
02de561
to
57ff0b9
Compare
b9ac5d3
to
a12c233
Compare
57ff0b9
to
5145f94
Compare
a12c233
to
9b98357
Compare
5145f94
to
c20e41a
Compare
This change is