mikamikasuki opened a new pull request, #44967:
URL: https://github.com/apache/superset/pull/44967

   ### SUMMARY
   Fixes #44912. Isoline and isoband formatting deliberately leaves one numeric 
field undefined. Skip numeric conversion for undefined values so unused fields 
remain undefined and are omitted from serialized form data instead of becoming 
NaN/null.
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   N/A: no visible UI change.
   
   ### TESTING INSTRUCTIONS
   From superset-frontend:
   ```bash
   ./node_modules/.bin/jest 
src/explore/components/controls/ContourControl/ContourPopoverControl-unused-fields.test.tsx
 --runInBand --silent
   ```
   The two regression tests fail on the current master revision with NaN and 
pass with this change. `uvx pre-commit run` passes all applicable hooks.
   
   ### ADDITIONAL INFORMATION
   - [x] Has associated issue: Fixes #44912
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API


-- 
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]

Reply via email to