deepujain commented on PR #44556:
URL: https://github.com/apache/superset/pull/44556#issuecomment-5804107826
Addressed both EnxDev review notes in commit c313d0e0:
1. Tooltip: reverted to the original `t('Logarithmic axis')` description in
`controlPanel.tsx` (the `stack` warning carries the explanation now), and
dropped the new msgid from `messages.pot`.
2. Test: `controlPanel.test.ts` now pins the warning text via
`toMatchObject({ disabled, warning: logAxis ? expect.any(String) : null })`,
verified against the exact `mapStateToProps` formula for all four truth-table
rows.
Fresh CI is running on the new head.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
