Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

fix-issue-3515 #3532

Merged
merged 3 commits into from
Sep 16, 2024
Merged

fix-issue-3515 #3532

merged 3 commits into from
Sep 16, 2024

Conversation

mohsin363
Copy link
Contributor

Pull request (PR) checklist

Please check if your pull request fulfills the following requirements:

  • I've read the Contribution Guidelines and my PR doesn't break the rules.
  • I've read and understand the Developer Guidelines.
  • I confirm that I've run the code locally and everything works as expected.
  • My PR includes only the necessary changes to fix the issue (i.e., no unnecessary files or lines of code are changed).
  • 🎬 I've attached a screen recording of using the new code to the next paragraph (if applicable).

Screen recording of your changes (if applicable):

What's changed?

Describe with a few bullets what's new:

  • I've fixed [BUG] Exchange Rate Clear Text Visibility Issue

Risk factors

What may go wrong if we merge your PR?

  • No know case

In what cases won't your code work?

  • No know case

Does this PR close any GitHub issues? (do not delete)

Troubleshooting GitHub Actions (CI) failures ❌

Pull request checks failing? Read our CI Troubleshooting guide.

@ILIYANGERMANOV ILIYANGERMANOV added the ready to merge 🚀 PR can be merged after the CI pass label Sep 16, 2024
@ILIYANGERMANOV
Copy link
Collaborator

@mohsin363 https://github.com/Ivy-Apps/ivy-wallet/blob/main/docs/CI-Troubleshooting.md

@shamim-emon
Copy link
Member

@mohsin363 paparazzi screenshot tests check is failing. Please refer to CI Troubleshooting guide if needed.

@ILIYANGERMANOV ILIYANGERMANOV merged commit f2c3795 into Ivy-Apps:main Sep 16, 2024
9 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready to merge 🚀 PR can be merged after the CI pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Exchange Rate Clear Text Visibility Issue
3 participants