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

Hide the developer mode from Tchap #367

Merged
merged 11 commits into from
Jan 21, 2022
Merged

Hide the developer mode from Tchap #367

merged 11 commits into from
Jan 21, 2022

Conversation

Claire1817
Copy link
Contributor

@github-actions
Copy link

github-actions bot commented Jan 19, 2022

Unit Test Results

11 files  ±0  11 suites  ±0   6s ⏱️ ±0s
83 tests ±0  83 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit f22da09. ± Comparison against base commit 1b2e050.

♻️ This comment has been updated with latest results.

@Florian14
Copy link
Contributor

Your implementation is interesting, until now, we managed these cases by adding a boolean in the app build.gradle to update the BuildConfig. Not sure one is better than the other, assigned to @yostyle to get his feedback.

@giomfo
Copy link
Contributor

giomfo commented Jan 20, 2022

Note: We will remove the Secure variant #358. The suggested changes here will have to be updated then. This is not a problem, I just highlight the point

Copy link
Contributor

@Florian14 Florian14 left a comment

Choose a reason for hiding this comment

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

SGTM

@Claire1817 Claire1817 merged commit be7e1a4 into develop Jan 21, 2022
@Claire1817 Claire1817 deleted the cgizard/ISSUE-359 branch January 21, 2022 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants