refactor(react-components): Change type of uniqueId and buttonType in the entire code

[nilscognite]

Type of change

Description 📝

1. Now, the uniqueId is made from Guid (before it was made from a counter)
2. Let buttonType return ByttonType type instead of string
3. Move ButtonType and UniqueId to architecture/utility/types.ts

How has this been tested? 🔍

I relay on the unit tests

Checklist ☑️

• I am happy with this implementation.
• I have performed a self-review of my own code.
• I have added unit and visuals tests to prove that my feature works to the best of my ability.
• I have added documentation to new and changed elements; both public and internally shared ones
• I have refactored the code for testability, readability and extendibility to the best of my ability.
• I have listed the JIRA tasks covering remaining work or tech debt related to this PR in the description.

[codecov]

Codecov Report

❌ Patch coverage is 97.36842% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 63.85%. Comparing base (d7b7631) to head (f0617f7).
⚠️ Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
...nts/src/components/Architecture/CommandButtons.tsx	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #5249      +/-   ##
==========================================
- Coverage   63.85%   63.85%   -0.01%     
==========================================
  Files        1170     1171       +1     
  Lines       82508    82503       -5     
  Branches     7404     7401       -3     
==========================================
- Hits        52685    52680       -5     
  Misses      29682    29682              
  Partials      141      141              

Files with missing lines	Coverage Δ
...ents/src/architecture/base/commands/BaseCommand.ts	94.61% <100.00%> (-0.05%)	⬇️
...base/concreteCommands/DeleteDomainObjectCommand.ts	100.00% <100.00%> (ø)
...rc/architecture/base/domainObjects/DomainObject.ts	89.35% <100.00%> (-0.02%)	⬇️
...chitecture/base/renderTarget/RevealRenderTarget.ts	61.70% <100.00%> (ø)
...mponents/src/architecture/base/undo/Transaction.ts	84.00% <ø> (ø)
...mponents/src/architecture/base/undo/UndoManager.ts	76.74% <100.00%> (ø)
...omponents/src/architecture/base/utilities/types.ts	100.00% <100.00%> (ø)
...tion360/DeleteSelectedImage360AnnotationCommand.ts	100.00% <100.00%> (ø)
...e/annotations/commands/AnnotationsDeleteCommand.ts	33.33% <100.00%> (ø)
...ure/concrete/clipping/commands/ApplyClipCommand.ts	27.77% <100.00%> (ø)
... and 11 more

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[nilscognite]

Changed to protected. This should be used with care, so it is better

[nilscognite]

Added this so it is done in the same way for both cases where I use UniqueId

[nilscognite]

This add readability in the code.

[nilscognite]

This is not needed anymore

[nilscognite]

Not needed anymore

[christjt]

risk review ok