remove workflowFinishState

[priyankakmEaton]

Fixes #5681

Changes proposed in this Pull Request:

• Remove the WorkflowCardFinishState component — this is an unnecessary wrapper component
• Move the EmptyState inline on the success screens
• add EmptyState props to successScreens

Screenshots / Screen Recording (if applicable)

To Test:

Any specific feedback you are looking for?

[github-actions]

Visit the preview URL for this PR (updated for commit 3fdd34b):

https://blui-react-login--pr577-bug-5681-removeworkf-zb2b943l.web.app

_{(expires Fri, 26 Apr 2024 12:13:47 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 1e9064d2e35ed37fe01e053587ea5f209719a043}

[JeffGreiner-eaton]

When last screen of registration loads (success screen) we throw error here

[ektaghag-eaton]

We need to update ForgotPasswordScreen, ResetPasswordScreen etc.. similar to https://github.com/etn-ccis/blui-react-native-workflows/pull/444/files#diff-0add99f28485696a5d5a9a93437d776378508625c2a7ecb1e58f7f13edf50e9d

[ektaghag-eaton]

One last change - we need to update docs where we have used this props.

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 72.38%. Comparing base (8bb6570) to head (8cbece5).
Report is 7 commits behind head on dev.

❗ Current head 8cbece5 differs from pull request most recent head 3fdd34b. Consider uploading reports for the commit 3fdd34b to get more accurate results

Files	Patch %	Lines
...ow/src/screens/SuccessScreen/SuccessScreenBase.tsx	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #577      +/-   ##
==========================================
+ Coverage   72.35%   72.38%   +0.02%     
==========================================
  Files          83       82       -1     
  Lines        1049     1050       +1     
  Branches      234      234              
==========================================
+ Hits          759      760       +1     
  Misses        120      120              
  Partials      170      170              

Flag	Coverage Δ
unittests	72.38% <66.66%> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[JeffGreiner-eaton]

This looks good. Do we know why the unit test coverage dropped for EULA screen or the difference in codecov patch being 66% and dev at 72%?

[surajeaton]

Other things look good.

awaiting