fix(DataTable): add sorting null values

[jsehull]

Closes #

Fix for Issue #12696

Changelog

New

• fix ability to sort data even if it has null values in rows

Changed

• n/a

Removed

• n/a

Testing / Reviewing

My basic findings BEFORE fix

• A: single null value in one column did NOT break sorting
• B: multiple null values in one column DID break sorting

1. open packages/react/src/components/DataTable/stories/shared.js
2. under const rows set various values to null (i.e. multiple name keys set to null for column sorting)
3. open the Storybook site to test in browser: Storybook > DataTable > Sorting > Default (or anywhere that has sorting enabled)
4. test the sort feature: hover over column headers and cycle clicks at least 3 times (3 different sorting comparison types), but also a bunch of times 💪
5. confirm both A and B findings (above) sort as expected now

[github-actions]

DCO Assistant Lite bot: Thanks for your submission! We ask that you sign our Developer Certificate of Origin before we can accept your contribution. You can sign the DCO by adding a comment below using this text:

---

I have read the DCO document and I hereby sign the DCO.

---

Jesse Hull seems not to be a GitHub user. You need a GitHub account to be able to sign the DCO. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You can retrigger this bot by commenting recheck in this Pull Request}

[jsehull]

I have read the DCO document and I hereby sign the DCO.

[netlify]

✅ Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	88b3fae
🔍 Latest deploy log	https://app.netlify.com/sites/carbon-components-react/deploys/63ecfeca634cb200085479d6
😎 Deploy Preview	https://deploy-preview-13155--carbon-components-react.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[netlify]

✅ Deploy Preview for carbon-elements ready!

Name	Link
🔨 Latest commit	88b3fae
🔍 Latest deploy log	https://app.netlify.com/sites/carbon-elements/deploys/63ecfecaf1361e0008b8f23e
😎 Deploy Preview	https://deploy-preview-13155--carbon-elements.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[jsehull]

@francinelucca same issue as before for DCO, I know what it is, old company email got removed and ended up as a default. I updated it but switched branches or something and it went back to the default.

Maybe I need to recreate this whole PR again. Thoughts?

[francinelucca]

Hey @jsehull 👋🏼
sorry, I think your best bet here will be to close out and do a new PR with the correct git configurations in place, it's the same thing as last time like you said.

[jsehull]

NEW PR created to address DCO error: #13161

[tw15egan]

Closing in favor of #13161