Skip to content

Optibp master merge#916

Open
junaidwarsivd wants to merge 15 commits intooptibp_mainfrom
optibp-master-merge
Open

Optibp master merge#916
junaidwarsivd wants to merge 15 commits intooptibp_mainfrom
optibp-master-merge

Conversation

@junaidwarsivd
Copy link
Copy Markdown
Contributor

No description provided.

@junaidwarsivd junaidwarsivd requested review from SebaMutuku, dubdabasoduba and hamza-vd and removed request for SebaMutuku, dubdabasoduba and hamza-vd November 4, 2022 08:27
@@ -82,7 +94,13 @@ public boolean isSyncSettings() {

@Override
public SyncFilter getEncryptionParam() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add tests for this method.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added the test

@@ -57,7 +62,14 @@ public String getExtraStringSettingsParameters() {
public String getSyncFilterValue() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Update or add tests for this method

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added the test

@@ -25,7 +25,12 @@ public int getSyncMaxRetries() {

@Override
public SyncFilter getSyncFilterParam() {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Update the tests for this method.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added the test

android:screenOrientation="portrait"
android:theme="@style/AncAppTheme.Login"
android:windowSoftInputMode="stateAlwaysHidden|adjustResize">
android:windowSoftInputMode="stateAlwaysHidden|adjustResize"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do we need this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this option is for the keyboard popup it was added to not automatically open the key for the text field which is not focused during the opening of the form

Comment thread opensrp-anc/build.gradle Outdated
exclude group: 'io.ona.rdt-capture', module: 'lib'
}
implementation('org.smartregister:opensrp-client-core:4.3.26-SNAPSHOT@aar') {
implementation('org.smartregister:opensrp-client-core:6.0.0-BETA3-SNAPSHOT@aar') {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

use 6.0.0-SNAPSHOT for client-core

Comment thread reference-app/build.gradle Outdated
}

implementation('org.smartregister:opensrp-client-core:4.3.26-SNAPSHOT@aar') {
implementation('org.smartregister:opensrp-client-core:6.0.0-BETA3-SNAPSHOT@aar') {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

use 6.0.0-SNAPSHOT for client-core

SebaMutuku
SebaMutuku previously approved these changes Nov 4, 2022
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