Skip to content

Conversation

JosephSanjaya
Copy link
Collaborator

@JosephSanjaya JosephSanjaya commented Aug 13, 2025

We try to add some basic guidance here.
Here is a great examples from the past @JosephSanjaya :
#89
#86

Copy link
Collaborator

👀🔎

Copy link
Collaborator

@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.

Tests did not run.

file:///Users/kerrywashington/Documents/grunt-software-projects/gruntsoftware/android/app/src/test/java/com/brainwallet/data/source/RemoteConfigSourceFirebaseImplTest.kt:20:49 Unresolved reference 'FirebaseImpl'.)

Crash Screenshot
Screenshot 2025-08-16 at 17 09 26

@@ -0,0 +1,63 @@
package com.brainwallet.data.repository
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes im already done for the remote config, but currently there's no key that use the source yet

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

on it to fix

@kcw-grunt
Copy link
Collaborator

@JosephSanjaya Here are my notes

@JosephSanjaya JosephSanjaya force-pushed the js/feat/#195-koin-annotations branch from 0c9fc8c to 314e401 Compare August 17, 2025 08:53
@JosephSanjaya
Copy link
Collaborator Author

{AD0F02C7-D8FD-4B78-A218-CF6410228030} fix the issue can you recheck the PR?

@kcw-grunt
Copy link
Collaborator

{AD0F02C7-D8FD-4B78-A218-CF6410228030} fix the issue can you recheck the PR?

Let's see a new PR from your clone @JosephSanjaya to make sure the Checks are triggered

@kcw-grunt
Copy link
Collaborator

Closing as we merged the cloned PR @JosephSanjaya

@kcw-grunt kcw-grunt closed this Aug 18, 2025
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.

2 participants