Commit 89d5d6c
authored
chore: remove unused QuickActionButtons temp component (#29845)
## **Description**
Removes the unused temporary QuickActionButtons component from
`app/component-library/components-temp/QuickActionButtons`.
This also removes the nested QuickActionButton component, which was only
used by QuickActionButtons, and removes stale Storybook registration
entries for both stories.
## **Changelog**
CHANGELOG entry: null
## **Related issues**
Fixes:
## **Manual testing steps**
```gherkin
Feature: temporary component cleanup
Scenario: QuickActionButtons temp component is removed
Given the repository is on this branch
When searching for QuickActionButtons temp component references
Then no app, test, or Storybook references are found
```
## **Screenshots/Recordings**
N/A. Code cleanup only.
### **Before**
N/A
### **After**
N/A
## **Validation**
- `rg -n "components-temp/QuickActionButtons|QuickActionButtons" app
tests .storybook --glob '!node_modules/**'` returned no matches.
## **Pre-merge author checklist**
- [ ] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I've included tests if applicable
- [x] I've documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] I've applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.
#### Performance checks (if applicable)
- [ ] I've tested on Android
- [ ] I've tested with a power user scenario
- [ ] I've instrumented key operations with Sentry traces for production
performance metrics
## **Pre-merge reviewer checklist**
- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
<!-- CURSOR_SUMMARY -->
---
> [!NOTE]
> **Low Risk**
> Low risk cleanup that only deletes an unused temporary component and
its Storybook/tests; primary risk is breaking any hidden imports if they
existed.
>
> **Overview**
> Removes the temporary `components-temp/QuickActionButtons`
implementation (including the nested `QuickActionButton`), along with
their associated unit tests and Storybook stories.
>
> Updates the auto-generated `.storybook/storybook.requires.js` to drop
the stale story registrations for these removed stories.
>
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
8ba4b53. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->1 parent a9e32b5 commit 89d5d6c
11 files changed
Lines changed: 0 additions & 667 deletions
File tree
- .storybook
- app/component-library/components-temp/QuickActionButtons
- QuickActionButton
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 0 additions & 25 deletions
This file was deleted.
Lines changed: 0 additions & 58 deletions
This file was deleted.
Lines changed: 0 additions & 32 deletions
This file was deleted.
Lines changed: 0 additions & 24 deletions
This file was deleted.
Lines changed: 0 additions & 2 deletions
This file was deleted.
Lines changed: 0 additions & 195 deletions
This file was deleted.
0 commit comments