change: Use loading icon to indicate tax id is being verified

[jaalah-akamai]

Description 📝

We need to verify the tax id before any of the events tax_id_valid or tax_id_invalid are fired off. This new icon provides a better UX.

See for more details: https://github.com/linode/manager/pull/10558

Changes 🔄

• Removed conditional icon for tax id

Target release date 🗓️

9/16

Preview 📷

Include a screenshot or screen recording of the change

https://github.com/user-attachments/assets/e2a9c20d-54ae-446b-a853-e4148c41973b

Before	After

How to test 🧪

Verification steps

Just observe that removing code does not break anything within the file itself.

As an Author I have considered 🤔

Check all that apply

• [x] 👀 Doing a self review
• [ ] ❔ Our contribution guidelines
• [ ] 🤏 Splitting feature into small PRs
• [x] ➕ Adding a changeset
• [ ] 🧪 Providing/Improving test coverage
• [ ] 🔐 Removing all sensitive information from the code and PR description
• [ ] 🚩 Using a feature flag to protect the release
• [ ] 👣 Providing comprehensive reproduction steps
• [ ] 📑 Providing or updating our documentation
• [ ] 🕛 Scheduling a pair reviewing session
• [ ] 📱 Providing mobile support
• [ ] ♿ Providing accessibility support

[github-actions[bot]]

Coverage Report: ✅
Base Coverage: 86.64%
Current Coverage: 86.64%

[jaalah-akamai]

Should we also clean up the tax id types/events/etc introduced in #10558?

We'll still need those 👍

[jaalah-akamai]

Does the API just clear the account's tax_id field if it is invalid? Wondering if it's bad that were just cleaning user data

It's not great and I made this known. There's a couple options we can do on the frontend to improve the experience while not displaying invalid tax ids on invoices. This will be coming soon, but until then we had to remove this one aspect since the requirements changed slightly and it was no longer accurate

[bnussman-akamai]

Understood! Thanks @jaalah-akamai