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

[Tech Debt] Remove stale api server code and add switch for dev mode - #98 #103

Closed
wants to merge 33 commits into from

Conversation

vsima
Copy link

@vsima vsima commented Jan 15, 2023

kcw-grunt and others added 25 commits May 24, 2022 17:51
- add uk xml file
-  removed the nl, sv, da files
- Escaped apostrphes in UK strings file
- Updated language choices
* Added notes

* Bugfix - add missing pendingIntent flag for android 13

* Use firebase bom

* subproject commit

* version bump

* Disabled FB send, Added Get Tasks

* Patch to hard code fees

- Static fees
- Removed the dep
- Turn crashlytics back on/off/on

Co-authored-by: Victor Sima <[email protected]>
* version bump

* Hotfix/crashes v2.6.0 (#17)

* Added notes

* Bugfix - add missing pendingIntent flag for android 13

* Use firebase bom

Co-authored-by: kcw-grunt <[email protected]>

Co-authored-by: Victor Sima <[email protected]>
* version code bump

* 🚀[ Release v.2.6.0] Merge into Main  (#12)

* Added український language

- add uk xml file
-  removed the nl, sv, da files
- Escaped apostrphes in UK strings file
- Updated language choices

* 🥳[Feature] Unstoppable ext update (#11)

* WIP: DNC

Dropped in the UD code prior to testing

* Added study link

* Fix unstoppable domains dependency

* Updated the UD ext text

WIP::::Need to refactor once UK and deletion on SV, NL, DA

* added UK lang

Co-authored-by: Victor Sima <[email protected]>

* version bump

* Hotfix/crashes v2.6.0 (#17)

* Added notes

* Bugfix - add missing pendingIntent flag for android 13

* Use firebase bom

Co-authored-by: kcw-grunt <[email protected]>

Co-authored-by: Victor Sima <[email protected]>

Co-authored-by: Victor Sima <[email protected]>
* added bitrefill

- added kotlin fix

* Added localizations

- added logo

* Added Bitrefill Link

* hides fiat scollview from Bitrefill

* Adjusted the padding at the buy tab item view

* fixe test in Bitrefill detail

* fied logo and layout

* bump version and version code
* added new logo

- reset the layout

* version code
* replaced coin logo

* bump versioncode

* fixed dep version

* added Bitrefill web listener
* Added the equivalenmt events

- updated web link

* WIP : Should compile

* Fixed constants

* Udpaded all Analytics constants

* Added basic tests

- This is WIP
- Need to debug the Tests

* version bump

* fiexd compile bug
@vsima vsima requested a review from kcw-grunt January 15, 2023 00:54
@vsima vsima self-assigned this Jan 15, 2023
@vsima vsima force-pushed the feature/api-server-98 branch from 58d2c50 to 332315e Compare January 15, 2023 00:58
kcw-grunt and others added 2 commits January 15, 2023 11:53
* 🚀[ Release v.2.7.0] Merge into Main (#30)

* Added український language

- add uk xml file
-  removed the nl, sv, da files
- Escaped apostrphes in UK strings file
- Updated language choices

* 💣🐞[Hotbug fix] sharedprefs fb fix (#19)

* Added notes

* Bugfix - add missing pendingIntent flag for android 13

* Use firebase bom

* subproject commit

* version bump

* Disabled FB send, Added Get Tasks

* Patch to hard code fees

- Static fees
- Removed the dep
- Turn crashlytics back on/off/on

Co-authored-by: Victor Sima <[email protected]>

* 🚀[ Release v.2.6.0] Merge into Develop  (#13)

* version bump

* Hotfix/crashes v2.6.0 (#17)

* Added notes

* Bugfix - add missing pendingIntent flag for android 13

* Use firebase bom

Co-authored-by: kcw-grunt <[email protected]>

Co-authored-by: Victor Sima <[email protected]>

* 🚀[Release] v2.6.1 (#21)

* version code bump

* 🚀[ Release v.2.6.0] Merge into Main  (#12)

* Added український language

- add uk xml file
-  removed the nl, sv, da files
- Escaped apostrphes in UK strings file
- Updated language choices

* 🥳[Feature] Unstoppable ext update (#11)

* WIP: DNC

Dropped in the UD code prior to testing

* Added study link

* Fix unstoppable domains dependency

* Updated the UD ext text

WIP::::Need to refactor once UK and deletion on SV, NL, DA

* added UK lang

Co-authored-by: Victor Sima <[email protected]>

* version bump

* Hotfix/crashes v2.6.0 (#17)

* Added notes

* Bugfix - add missing pendingIntent flag for android 13

* Use firebase bom

Co-authored-by: kcw-grunt <[email protected]>

Co-authored-by: Victor Sima <[email protected]>

Co-authored-by: Victor Sima <[email protected]>

* Feature/add bitrefill (#28)

* added bitrefill

- added kotlin fix

* Added localizations

- added logo

* Added Bitrefill Link

* hides fiat scollview from Bitrefill

* Adjusted the padding at the buy tab item view

* fixe test in Bitrefill detail

* fied logo and layout

* bump version and version code

* Feature/update branding (#29)

* added new logo

- reset the layout

* version code

* replaced coin logo

* bump versioncode

* fixed dep version

* added Bitrefill web listener

Co-authored-by: Victor Sima <[email protected]>

* Create FUNDING.yml

For GH Sponsor

* 💣🐞[Hotbug fix] sharedprefs fb fix (#19)

* Added notes

* Bugfix - add missing pendingIntent flag for android 13

* Use firebase bom

* subproject commit

* version bump

* Disabled FB send, Added Get Tasks

* Patch to hard code fees

- Static fees
- Removed the dep
- Turn crashlytics back on/off/on

Co-authored-by: Victor Sima <[email protected]>

* Feature/add bitrefill (#28)

* added bitrefill

- added kotlin fix

* Added localizations

- added logo

* Added Bitrefill Link

* hides fiat scollview from Bitrefill

* Adjusted the padding at the buy tab item view

* fixe test in Bitrefill detail

* fied logo and layout

* bump version and version code

* Feature/update branding (#29)

* added new logo

- reset the layout

* version code

* 🚀[ Release v.2.7.0] Merge into Develop (#32)

* replaced coin logo

* bump versioncode

* fixed dep version

* added Bitrefill web listener

* [Techdebt/] pdate match firebase events 33 (#48)

* Added the equivalenmt events

- updated web link

* WIP : Should compile

* Fixed constants

* Udpaded all Analytics constants

* Added basic tests

- This is WIP
- Need to debug the Tests

* version bump

* fiexd compile bug

* version bump

* renmoved staging

* moved backup rates

* code bump

* Updated test configs

* Move api resources to central location

- version code bump

Co-authored-by: Victor Sima <[email protected]>
kcw-grunt and others added 2 commits January 15, 2023 12:18
* Feature - add donation modal disclaimer
 - #60

* Add missing strings

* Upgraded AS

- Reduced donation amount
- Renamed id WIP

* updated stirings key format

- updated the donation  address

* code bump

Co-authored-by: kcw-grunt <[email protected]>
@vsima vsima marked this pull request as draft January 17, 2023 16:33
@vsima vsima force-pushed the feature/api-server-98 branch 3 times, most recently from e06ef90 to e013c19 Compare January 17, 2023 20:55
@vsima vsima marked this pull request as ready for review January 17, 2023 20:55
@vsima vsima force-pushed the feature/api-server-98 branch from e013c19 to 49bea1f Compare January 17, 2023 21:05
Copy link
Contributor

@kcw-grunt kcw-grunt left a comment

Choose a reason for hiding this comment

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

We are really close. When I tested it I realized there is no direct way the non-tech users will know which mode they are in.
It needs to be a big flag for them and our support guys to aid in troubleshotting.

This label should disappear when they are in prod mode and when prod server is down, the client should auto-switch to dev and all labels are updated.

Will you add this label in the Nav bar?:

Dev Mode Label
Main with Dev mode on

public String title;
public String addonText;
public Boolean isChecked;
public View.OnClickListener listener;

Copy link
Contributor

Choose a reason for hiding this comment

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

This is a really good start. We need to make sure it's visible. Will you add this:

  • Status label
  • Switch background color
Settings Dev Mode: On Settings Dev Mode: Off
Developer Mode_ Switch Developer Mode_ Switch Off

Copy link
Author

Choose a reason for hiding this comment

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

How does this look?

Copy link
Contributor

Choose a reason for hiding this comment

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

@vsima vsima force-pushed the feature/api-server-98 branch 2 times, most recently from 684bcb6 to 35f8e90 Compare January 21, 2023 01:04
@vsima
Copy link
Author

vsima commented Jan 21, 2023

Dev mode copy added to the main screen:

@kcw-grunt kcw-grunt self-requested a review January 21, 2023 10:07
@kcw-grunt
Copy link
Contributor

cc: @vsima the servers are not quite ready so I approved but not merging until after the
#82 is all ready and we get a prod release out.

@kcw-grunt kcw-grunt added this to the v2.9.0 milestone Jan 21, 2023
vsima and others added 4 commits January 24, 2023 23:40
* Feature - Show Litecoin Card as Webview
 - remove partner api library
 - create card frament webview and add to bottom nav
 - fixes #58

* bump version code

* Fix UI crash

* bump version code

Co-authored-by: kcw-grunt <[email protected]>
 - move dev & prod urls into constants
Copy link
Contributor

@josikie josikie left a comment

Choose a reason for hiding this comment

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

Great! @kcw-grunt @vsima
Thanks!

I have some questions:

  1. what is this syntax called: v -> {....} (AdvancedActivity.java, lines 141)?
  2. what is an inflater?
  3. what is android:foreground for, and why does it value selectableItemBackground?

@kcw-grunt
Copy link
Contributor

will not do

@kcw-grunt kcw-grunt closed this Mar 29, 2024
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.

🦺[Tech Debt] Remove stale api server code and add switch for dev mode
3 participants