[BpkInfoBanner] Changed typing declarations to match common-types.ts file

[sambocharov]

Changed typing declarations in common-types.d.ts to match common-types.ts

Remember to include the following changes:

• README.md (If you have created a new component)
• Component README.md
• Tests
• Storybook examples created/updated
• For breaking changes or deprecating components/properties, migration guides added to the description of the PR. If the guide has large changes, consider creating a new Markdown page inside the component's docs folder and link it here

[github-actions]

	Warnings
⚠️	Package source files (e.g. packages/package-name/src/Component.js) were updated, but snapshots weren't. Have you checked that the tests still pass?

Browser support

If this is a visual change, make sure you've tested it in multiple browsers.

Generated by 🚫 dangerJS against 943baea

[github-actions]

Visit https://backpack.github.io/storybook-prs/3094 to see this build running in a browser.

[olliecurtis]

This value would always have a value right as we set a default in the Inner Banner should would never need to be optional here and may also need to be updated in the common-types file too?

[sambocharov]

I hope I get it right, but won't BpkInfoBanner complain about typing if we'll remove optional here? I mean, we may not pass to BpkInfoBanner a type prop and it should still work and pass typechecking. Since BpkInfoBanner has CommonProps as type for props, if we'll make type required it would fail. According to figma, if I get it right, it has a default value which we specify in BpkInfoBannerInner, as you mentioned, and type marked as not required.

[olliecurtis]

Ah yeah you are right my mistake here, apologies!

[github-actions]

Visit https://backpack.github.io/storybook-prs/3094 to see this build running in a browser.