fix: consolidate person view types

[gavinbarron]

Closes #2941

PR Type

• Bugfix

Description of the changes

removes the PersonViewType enum in favor of the ViewType enum

BREAKING CHANGE: PersonViewType has been removed. 'avatar' is no longer a valid value for the 'view' attribute on the 'mgt-person' component

PR checklist

• Project builds (yarn build) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)
• All public APIs (classes, methods, etc) have been documented following the jsdoc syntax
• Stories have been added and existing stories have been tested
• Added appropriate documentation. Docs PR: https://github.com/microsoftgraph/microsoft-graph-docs/pull/23336

• License header has been added to all new source files (yarn setLicense)
• Contains NO breaking changes

Other information

Missing docs.

[sebastienlevert]

I'd love to see a docs PR for this one.

[gavinbarron]

Interestingly as I looked at this more closely using PersonViewType.avatar only worked due to an overlap of the enum numeric values between it and ViewType.image. Providing the string "avatar" to the view attribute only worked because of ViewType.image being the default when the supplied string couldn't be correctly converted to a ViewType.

This change is essentially eliminate a duplicate type that can cause confusion.

[github-actions]

Package	Line Rate	Branch Rate	Complexity	Health
mgt-components.src.components	80%	100%	0	✔
mgt-components.src.components.mgt-contact	63%	100%	0	➖
mgt-components.src.components.mgt-file	51%	100%	0	➖
mgt-components.src.components.mgt-file-list	56%	100%	0	➖
mgt-components.src.components.mgt-file-list.mgt-file-upload	49%	88%	0	❌
mgt-components.src.components.mgt-get	30%	100%	0	❌
mgt-components.src.components.mgt-messages	66%	100%	0	➖
mgt-components.src.components.mgt-organization	47%	100%	0	❌
mgt-components.src.components.mgt-person	81%	64%	0	✔
mgt-components.src.components.mgt-person-card	65%	50%	0	➖
mgt-components.src.components.mgt-picker	78%	100%	0	✔
mgt-components.src.components.mgt-profile	40%	100%	0	❌
mgt-components.src.components.mgt-tasks-base	77%	100%	0	✔
mgt-components.src.components.mgt-theme-toggle	100%	100%	0	✔
mgt-components.src.components.mgt-todo	79%	100%	0	✔
mgt-components.src.components.sub-components.mgt-arrow-options	71%	100%	0	➖
mgt-components.src.components.sub-components.mgt-flyout	78%	53%	0	✔
mgt-components.src.components.sub-components.mgt-spinner	100%	100%	0	✔
mgt-components.src.graph	39%	89%	0	❌
mgt-components.src.styles	92%	80%	0	✔
mgt-components.src.utils	79%	19%	0	✔
mgt-element.dist.es6.components.src.components	73%	80%	0	➖
mgt-element.dist.es6.mock.src.mock	90%	72%	0	✔
mgt-element.dist.es6.providers.src.providers	85%	73%	0	✔
mgt-element.dist.es6.src	91%	80%	0	✔
mgt-element.dist.es6.utils.src.utils	66%	69%	0	➖
mgt-element.src	93%	40%	0	✔
mgt-element.src.components	78%	100%	0	✔
mgt-element.src.mock	81%	56%	0	✔
mgt-element.src.providers	83%	91%	0	✔
mgt-element.src.utils	71%	90%	0	➖
Summary	66% (14249 / 21595)	64% (429 / 667)	0	➖