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

Show the Test screen only in the DEBUG build type #2524

Merged
merged 1 commit into from
Sep 1, 2023

Conversation

mohammednawas8
Copy link
Contributor

Pull Request (PR) Checklist

Please check if your pull request fulfills the following requirements:

  • The PR is submitted to the main branch.
  • I've read the Contribution Guidelines.
  • The code builds and is tested on an actual Android device.
  • I confirm that I've run the code locally and everything works as expected.

Put an x in the boxes that apply.

  • Demo: Checking checkbox using [x]

Pull Request Type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, new lines, etc.)
  • Refactoring (no functional changes, renaming)
  • Small improvement (fix typo, UI fine-tune, change color or something small)
  • Gradle Build related changes
  • Dependencies update (updating libraries)
  • Documentation (clarifying comments, KDoc)
  • Tests (Unit, Integration, UI tests)
  • Other (please describe):

Put an x in all the boxes that apply.

Does this PR closes any GitHub Issues? yes

Check Ivy Wallet Issues.

  • Closes #N/A (2523)

What's changed?

Describe with a few bullets what's new:

  • a Check if the app is in the DEBUG Version when the App's version text gets clicked by BuildConfig.DEBUG .
  • b Navigate to the test screen only when BuildConfig.DEBUG is true

💡Tip: Attach screenshots and screen recordings. It helps a lot!

@mohammednawas8 mohammednawas8 changed the title Show the Test screen only in the DEBUG build type (#2523) Show the Test screen only in the DEBUG build type Sep 1, 2023
@ILIYANGERMANOV ILIYANGERMANOV self-requested a review September 1, 2023 12:49
Copy link
Collaborator

@ILIYANGERMANOV ILIYANGERMANOV left a comment

Choose a reason for hiding this comment

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

LGTM!

@ILIYANGERMANOV ILIYANGERMANOV merged commit 62d791b into Ivy-Apps:main Sep 1, 2023
@mohammednawas8 mohammednawas8 deleted the issue-2523 branch September 6, 2023 19:25
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