Skip to content
This repository has been archived by the owner on Jun 16, 2022. It is now read-only.

[LIVE-2320] - Bugfix: Fix missing import for Clipboard #2503

Merged
merged 2 commits into from
May 11, 2022

Conversation

lambertkevin
Copy link
Contributor

LIVE-2320

⚠️ FYI: When tested in the iOS simulator, it's still crashing the app... I had to use a real device to actually test it without crash.

Type

Bugfix

Context

RPReplay_Final1652259395.mov
clipboard.mp4

Parts of the app affected / Test plan

Go to NFT Viewer & click on any "copy" button

@lambertkevin lambertkevin requested a review from a team as a code owner May 11, 2022 09:20
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Code looks OK, pending QA approval.

@@ -17,6 +17,7 @@ import { BigNumber } from "bignumber.js";
import { useSelector } from "react-redux";
import { Button, Icons } from "@ledgerhq/native-ui";
import { useTranslation, Trans } from "react-i18next";
import Clipboard from "@react-native-community/clipboard";
Copy link

Choose a reason for hiding this comment

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

I'm wondering how it could build before without that import 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We don't have any CI anymore on TS files... So that + everything is red in VSCode cause we don't have either the proper configuration, or we didn't take the time to actually fix all of our errors, it makes it almost impossible to catch this kind of things...

@lambertkevin lambertkevin changed the title Bugfix: Fix missing import for Clipboard [LIVE-2320] - Bugfix: Fix missing import for Clipboard May 11, 2022
@lambertkevin lambertkevin force-pushed the bugfix/clipboard-crash branch 2 times, most recently from 9daffb8 to 0be6c23 Compare May 11, 2022 12:11
@lambertkevin lambertkevin force-pushed the bugfix/clipboard-crash branch from 0be6c23 to 78dc867 Compare May 11, 2022 12:41
Copy link

@ychen-ledger ychen-ledger left a comment

Choose a reason for hiding this comment

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

@lambertkevin lambertkevin merged commit ab2b638 into release/3.1.x May 11, 2022
@lambertkevin lambertkevin deleted the bugfix/clipboard-crash branch May 11, 2022 14:07
This was referenced May 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants