Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(js-ts): Convert app/components/UI/GenericButton/index.ios.js to TypeScript #11523

Conversation

devin-ai-integration[bot]
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Oct 1, 2024

Description

This PR converts the app/components/UI/GenericButton/index.ios.js file to TypeScript. The conversion involves updating the file extension to .tsx, defining TypeScript interfaces for props, and removing propTypes.

Motivation

The conversion to TypeScript is part of an ongoing effort to improve type safety and maintainability across the codebase.

Context

The GenericButton component is deprecated in favor of the new Button component from the component-library. This conversion ensures that the deprecated component is type-safe until it is fully replaced.

Related Issues

Manual Testing Steps

  1. Verify that the GenericButton component renders correctly in the app.
  2. Ensure that the component's props (children, style, onPress) function as expected.
  3. Check for any TypeScript errors in the component and related files.

Screenshots/Recordings

N/A

Additional Information

  • Followed MetaMask Coding Standards.
  • No semantic changes were made to the code.
  • Labels added: "needs-dev-review", "team-mobile-platform", "No QA Needed".

If you have any feedback, you can leave comments in the PR and I'll address them in the app!

Copy link
Contributor

github-actions bot commented Oct 1, 2024

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@github-actions github-actions bot added the team-ai AI team (for the Devin AI bot) label Oct 1, 2024
@devin-ai-integration devin-ai-integration bot added team-mobile-platform needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) skip-sonar-cloud Only used for bypassing sonar cloud when failures are not relevant to the changes. Run Smoke E2E Triggers smoke e2e on Bitrise labels Oct 1, 2024
Copy link
Contributor

github-actions bot commented Oct 1, 2024

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: d5f4383
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/da4a12eb-b5aa-4974-b5cb-2527e8cb5262

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@devin-ai-integration devin-ai-integration bot marked this pull request as ready for review October 1, 2024 09:12
@devin-ai-integration devin-ai-integration bot requested a review from a team as a code owner October 1, 2024 09:12
Copy link

sonarcloud bot commented Oct 1, 2024

Copy link
Contributor

@kylanhurt kylanhurt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kylanhurt kylanhurt added this pull request to the merge queue Oct 2, 2024
Merged via the queue into main with commit 4092f86 Oct 2, 2024
53 of 54 checks passed
@kylanhurt kylanhurt deleted the devin/convert-ts-app-components-UI-GenericButton-index-ios.js-9184 branch October 2, 2024 19:30
@github-actions github-actions bot locked and limited conversation to collaborators Oct 2, 2024
@github-actions github-actions bot removed the needs-dev-review PR needs reviews from other engineers (in order to receive required approvals) label Oct 2, 2024
@gauthierpetetin gauthierpetetin added the release-7.33.0 Issue or pull request that will be included in release 7.33.0 label Oct 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-7.33.0 Issue or pull request that will be included in release 7.33.0 Run Smoke E2E Triggers smoke e2e on Bitrise skip-sonar-cloud Only used for bypassing sonar cloud when failures are not relevant to the changes. team-ai AI team (for the Devin AI bot) team-mobile-platform
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants