Skip to content

[charts] Add test for print dialog export #17327

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

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

bernardobelchior
Copy link
Member

Adds a test for the functionality merged in #17285.

Copy link

github-actions bot commented Apr 11, 2025

Thanks for adding a type label to the PR! 👍

@bernardobelchior bernardobelchior added test component: charts This is the name of the generic UI component, not the React module! maintenance labels Apr 11, 2025
@bernardobelchior bernardobelchior force-pushed the charts-print-dialog-test branch 2 times, most recently from f6a8b76 to 0b682eb Compare April 11, 2025 14:55
@mui-bot
Copy link

mui-bot commented Apr 11, 2025

Deploy preview: https://deploy-preview-17327--material-ui-x.netlify.app/

Updated pages:

Generated by 🚫 dangerJS against dacdc76

Copy link

codspeed-hq bot commented Apr 11, 2025

CodSpeed Performance Report

Merging #17327 will not alter performance

Comparing bernardobelchior:charts-print-dialog-test (dacdc76) with master (24af71e)

Summary

✅ 8 untouched benchmarks

@bernardobelchior
Copy link
Member Author

Last working build (can't replicate it, though): 0a0422d

https://app.argos-ci.com/mui/mui-x/builds/34422/152017419

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 11, 2025
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@bernardobelchior bernardobelchior force-pushed the charts-print-dialog-test branch from da5314f to 4b4817d Compare April 14, 2025 08:03
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Apr 14, 2025
@bernardobelchior bernardobelchior force-pushed the charts-print-dialog-test branch from 4b4817d to 44b835d Compare April 14, 2025 09:00
@bernardobelchior bernardobelchior force-pushed the charts-print-dialog-test branch from 000d826 to dacdc76 Compare April 14, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: charts This is the name of the generic UI component, not the React module! maintenance test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants