Skip to content

#421: GoalTemplate for Steps - #65

Merged
westei merged 1 commit into
developfrom
umm/421-step-goal-template
Jul 8, 2026
Merged

#421: GoalTemplate for Steps#65
westei merged 1 commit into
developfrom
umm/421-step-goal-template

Conversation

@westei

@westei westei commented Jul 7, 2026

Copy link
Copy Markdown
Member

No description provided.

@westei
westei requested a review from iaigner July 7, 2026 13:52
@westei westei self-assigned this Jul 7, 2026
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

Test Results

215 tests  ±0   214 ✅ ±0   1m 48s ⏱️ -3s
 59 suites ±0     1 💤 ±0 
 59 files   ±0     0 ❌ ±0 

Results for commit f9faef2. ± Comparison against base commit 8a103c1.

@iaigner iaigner left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

kleines kommentar sonst schauts gut aus


public StepGoalTemplateFactory() { }

public static final String FIELD_TARGET_STEPS = "targetSteps";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

was mir da noch eingefallen ist. evtl brauchen wir ein toggle ob das ziel vom garmin genommen wird oder das ziel direkt eingegeben werden kann. glaub das haben wir noch nicht drinnen oder?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@janoliver20 hast Du das in der App implementiert?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ist ausgeredet. Die DAILY_STEPS daten von GarminConnect sind derzeit nicht verfügbar. Schauen erstmal warum und dann updaten wir das GoalTemplate

@westei
westei merged commit 9862671 into develop Jul 8, 2026
5 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