Skip to content

fix: [ANDROSDK-2272] Weekly start day setting name is wrong#2567

Merged
taridepaco merged 5 commits intodevelopfrom
ANDROSDK-2272
Mar 16, 2026
Merged

fix: [ANDROSDK-2272] Weekly start day setting name is wrong#2567
taridepaco merged 5 commits intodevelopfrom
ANDROSDK-2272

Conversation

@taridepaco
Copy link
Copy Markdown
Collaborator

@taridepaco taridepaco commented Mar 12, 2026

This task fixes an error in the weekly start setting name that prevented its download and use, defaulting to Monday starting weeks in every analysis. Some other related issues have been fixed:

  • fields request keyword for settings changed from "fields" to "key", thus returning only the desired ones instead of all of them
  • added tests for weekly relative periods
  • added weekly start day key to list of requested ones

Related task: ANDROSDK-2272

@taridepaco taridepaco self-assigned this Mar 12, 2026
val startDate = period.startDate()
assertThat(startDate).isNotNull()

val calendar = java.util.Calendar.getInstance()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you avoid using this class?

@sonarqubecloud
Copy link
Copy Markdown

@taridepaco taridepaco merged commit 937ae30 into develop Mar 16, 2026
10 checks passed
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