Skip to content

feat: [ANDROAPP-7605] Add granular sync use-case for datasets and data values#4957

Merged
Balcan merged 1 commit into
developfrom
ANDROAPP-7605-syncdataset
Jul 8, 2026
Merged

feat: [ANDROAPP-7605] Add granular sync use-case for datasets and data values#4957
Balcan merged 1 commit into
developfrom
ANDROAPP-7605-syncdataset

Conversation

@Balcan

@Balcan Balcan commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Description

Link the JIRA issue.

Please provide a clear definition of the problem and explain your solution.

.blockingUpload()
}

// TODO: The sdk does not provide methods to download data values for a given dataset

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

is there anything to do here or could this be removed?

@Balcan Balcan Jul 8, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The SDK lacks these methods, but they are important. I'll be opening a ticket in the sdk to add these methods so we can use them.

}

// TODO: The sdk does not provide methods to download data values for a given dataset
override suspend fun downloadDataSet(dataSetUid: DataSetUid) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why do you create this method if is empty? is it being used?

@Balcan Balcan force-pushed the ANDROAPP-7605-syncdataset branch from e6b2ae0 to 523072d Compare July 8, 2026 12:39
@sonarqubecloud

sonarqubecloud Bot commented Jul 8, 2026

Copy link
Copy Markdown

@Balcan Balcan requested a review from andresmr July 8, 2026 13:14
@Balcan Balcan merged commit f5f8fc7 into develop Jul 8, 2026
14 checks passed
@Balcan Balcan deleted the ANDROAPP-7605-syncdataset branch July 8, 2026 13:15
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.

3 participants