fix(protocol-designer): remove dropdown menu for module quantity

[koji]

Overview

remove dropdown menu for module quantity and do a little bit refactoring.
remove the snackbar because it doesn't hit the module limit.

SelectModules still doing a lot of things in the component.

Refactoring is necessary to separate the UI and logic to enhance readability and maintainability in the future.

close AUTH-1451

Test Plan and Hands on Testing

• go to select modules screen OT-2
• go to select modules screen Flex

Changelog

• move SelectModules to organisms and remove dropdown menu
• export AddModuleEmptySelectorButton as a component and add test

Review requests

Risk assessment

low

[codecov]

Codecov Report

Attention: Patch coverage is 58.82353% with 77 lines in your changes missing coverage. Please review.

Project coverage is 18.02%. Comparing base (875346d) to head (abdf845).
Report is 72 commits behind head on edge.

Files with missing lines	Patch %	Lines
...col-designer/src/organisms/SelectModules/index.tsx	50.64%	77 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             edge   #17531      +/-   ##
==========================================
- Coverage   26.17%   18.02%   -8.16%     
==========================================
  Files        3191     3090     -101     
  Lines      230174   222921    -7253     
  Branches     9814     7005    -2809     
==========================================
- Hits        60251    40178   -20073     
- Misses     169898   182743   +12845     
+ Partials       25        0      -25     

Flag	Coverage Δ
app	?
labware-library	?
opentrons-ai-client	?
protocol-designer	18.02% <58.82%> (+0.59%)	⬆️
shared-data	?
step-generation	?

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

Files with missing lines	Coverage Δ
...sms/SelectModules/AddModuleEmptySelectorButton.tsx	100.00% <100.00%> (ø)
...signer/src/pages/CreateNewProtocolWizard/index.tsx	0.00% <ø> (ø)
...col-designer/src/organisms/SelectModules/index.tsx	50.64% <50.64%> (ø)

... and 593 files with indirect coverage changes

[ncdiehl11]

nice

[jerader]

with this change, doesn't that mean we can simplify getNumSlotsAvailable?

[koji]

i still don't touch that part since the ticket only mentions to modules. i added my question to weekly meeting.
once i get an answer i will update getNumSlotsAvailable.

[jerader]

it works but i think there is additional cleanup to be done:

1. getNumSlotsAvailable can be simplified since we won't need to account for MoaM logic for modules anymore
2. the createModuleWithNoSlot thunk used in CreateNewProtocolWizard is never called anymore since the modules will always have an assigned slot associated with them. So i recommend deleting that thunk