[Bug] Fix duplicate heading roles in Alert component

[oddballdave]

Description of Change

Removed the role="heading" property from the Alert component as this is already specified in the child Text component by virtue of the variant="heading property.

Testing Packages

• 0.30.1-alpha.2

Screenshots/Video

Testing

Tested library separately. Also verified this allowed the VA Mobile App unit tests to function properly.

• Tested on iOS
• Tested on Android
• Tested on Web

PR Checklist

Code reviewer validation:

• General
  • PR is linked to ticket(s)
  • PR has changelog label applied if it's to be included in the changelog
  • Acceptance criteria:
    • All satisfied or
    • Documented reason for not being performed or
    • Split to separate ticket and ticket is linked by relevant AC(s)
  • Above PR sections adequately filled out
  • If any breaking changes, in accordance with the pre-1.0.0 versioning guidelines: a CU ticket has been created for the VA Mobile App detailing necessary adjustments with the package version that will be published by this ticket
• Code
  • Tests are included if appropriate (or split to separate ticket)
  • New functions have proper TSDoc annotations

Publish

If changes warrant a new version per the versioning guidelines and the PR is approved and ready to merge:

• Merge main into branch
• Merge branch to main
• Verify that Check Component Integrations workflow ran successfully
• Publish new version after merging

[DonMcCaugheyUSDS]

Nice, thank you!

[kimmccaskill-oddball]

Looks pretty straightforward thanks!

[Eallan919]

These PR's are blocked because we are not authorized to merge PR's to main in the DS Library: #617 & #621. We are also likely blocked on publishing/releasing the DS Library package, which happens after we merge.

[Eallan919]

These PR's are blocked because we are not authorized to merge PR's to main in the DS Library: #617 & #621. We are also likely blocked on publishing/releasing the DS Library package, which happens after we merge.