-
Notifications
You must be signed in to change notification settings - Fork 25
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
fix: Add Percy specific CSS to root resource after asset discovery #352
Conversation
af45aa3
to
1903da1
Compare
9846a90
to
97ccaf6
Compare
Before, it's possible the asset discovery resource saves the CSS as a different content type (for example, if their server just serves the root index for this request, we would get DOM instead of CSS)
97ccaf6
to
6e23317
Compare
b27038d
to
0d1cdca
Compare
rootResourceUrl, | ||
domSnapshot, | ||
options, | ||
logger, | ||
) | ||
} |
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.
This method no longer seems useful. Could leave it as an alias, or just call this.snapshotService.assetDiscoveryService.discoverResources
directly from the agent service.
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.
Yeah for sure, going to clean that up in future PRs
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.
4th review is the charm?
## [0.15.2](v0.15.1...v0.15.2) (2019-09-24) ### Bug Fixes * Add Percy specific CSS to root resource after asset discovery ([#352](#352)) ([5984a65](5984a65))
What is this?
While debugging some percy CSS weirdness I thought it would be pretty nice to see in the logs when this is created and if its skipped, why?