appcreativity / kliva Goto Github PK

To get rid of the new instances of HttpClient on every call.

• The side pane icons are not following the navigation correctly
• When you select a club to show it's members, you'll need to press the back button twice to get out of it again

• pace
• hr zones

http://timheuer.com/blog/archive/2015/05/06/making-circular-images-in-xaml-easily.aspx

We already set the width and height of our Ellipse but we need to pixel decode too!

I stopped during my ride today for a nice picture .. but neither Kliva or the Strava site currently support uploading a picture ... 😞

For views that you may not need a PivotItem for, they should be marked for defer loading and then un-deferred when needed. something like:

<Pivot>
  <PivotItem Header="Item 1" />
  <PivotItem Header="Item 2" x:DeferLoadStrategy="Lazy" x:Name="UnusedPivot" />
  <PivotItem Header="Item 3" />
</Pivot>

then code:

if (UnusedPivot == null) {
  FindName("UnusedPivot");
}
UnusedPivot.Visibility = Visibility.Visible;

Good pattern to use for items that may not always be visible in the UI.

We want to show of progress of goals for given athlete, the needed endpoint is described here:

http://strava.github.io/api/internal/v3/athlete/#goals

To show the list of athletes that participated in a given activity we need to query for all related activities based upon the activity id of a selected activity.

In the WinRT 8.1 app the API call for this is: string.Format("{0}/{1}/related{2}", Constants.STRAVA_ACTIVITY_ENDPOINT, id,...

But it seems this is missing from StravaDotNet.
So we need to add it in Kliva

Improve the overall performance of activity list feed

When you select 'my activities' as active filter and close / reopen the app, you'll get the activity feed of 'my activities'.

But the filter selection on top will not show this, it will point to Show all activities.

Secondly, loading saved activities through this, will no longer show athlete avatars?

We need to check if the current user is the same one as the current selected activity owner.
If so don't allow giving kudos!

The timestamp = await LocalCacheService.getCacheTimestamp(name); line needs to be moved out of the if (!_hasLoaded) block, so that the cache time is checked on refresh operations.

I removed my fork to get rid of the bad history, so I don't have a place to check in.

Total distance run with shoe

Right now the Activity detail screen already has the text controls set for all the statistics, without an activity being selected. This looks rather odd, but an empty screen isn't perfect either as this doesn't invite to select an activity.

A possible solution would be to show a blurred detail screen to invite the selection of an activity. Issue: this can't be an image if we want it to scale perfect to the real screen on each device.

Since the APIv3 does not allow retrieving photos for any athlete, the GetPhotos shouldn't be called unless that user. The API does tell you for ANY activity how many photos, but the check for the count is presently insufficient if you can't enumerate them. So right now an unnecessary call is being made to the API which will always return an empty array.

Images posted via linked Instragram work. Images posted from a strava client don't show up.

Followers list on profile page and Kudos/Comments list on Activity detail have different top margin in reference to pivot headers
Also check Top athletes on segment detail

Hi,

on the user stats page it displays total times in dd:hh:mm format. So if I cycled for 30 hours it will show 1:06:00, which is basically quite confusing as it could also mean 1 hour and 6 minutes.

Maybe you should "humanize" these values with a postfix, like 4d 10h 3m which is obvious and easy to understand.

Thx!

https://www.strava.com/api/v3/athletes/:id/segments/starred

When we navigate away from an activity detail page to the activity detail full map page and navigate back. We won't see the activity detail polygon on the map anymore.

The real issue is because we are no longer sending the data message anymore ( https://github.com/AppCreativity/Kliva/blob/master/src/Kliva/ViewModels/MainViewModel.cs#L66 ), but we need to see why the map get's cleared when returning!

-- Bug only on Phone

Would be great if we could swipe to quickly like an activity ( or unlike )

Add the same orange bar / header title that profile etc have to Activity view

Do we need to change the inactive titlebar color? Verify what the default color is and change if needed...

titleBar.InactiveBackgroundColor = titleBar.BackgroundColor = Windows.UI.Colors.Black;
titleBar.InactiveForegroundColor = titleBar.ForegroundColor = Windows.UI.Colors.White;
titleBar.ButtonInactiveBackgroundColor = titleBar.ButtonBackgroundColor = Windows.UI.Colors.Black;
titleBar.ButtonInactiveForegroundColor = titleBar.ButtonForegroundColor = Windows.UI.Colors.White

We've experienced several users having issues logging in to Strava. There are several possible reasons, sometimes it just worked by retrying to log in. We've also received following feedback from a user:

Regarding the issue that I was having with kliva I found that I could resolve it by matching my language and region on my phone. If I have my phone and English and the region as Mexico it does not let me log in which I suppose is related to the strava API.

Add an icon to view switcher for activity view
Fix navigation to what Jake is doing in his sample
Build empty views on view switcher hook up switching

The ability to filter the activity list to show just my activities vs other peoples.

When requesting Activities we currently don't get full detailed information.
Meaning that the Photo property isn't filled in, so we validate if we need to retrieve extra photo data by checking the photo_count field.
Seems that Strava isn't fail prove on this! When we get photos that aren't stored on Instagram the photo_count field will show 0 even though that when we request the photo data, we get Photos.Count that is higher than 0.

We need to check if we can circumvent this problem somehow

Strava api calls are limited in time

• In memory vs on disk (eg Akavache)
• Different timespans as some endpoints expire faster (eg kudos/comments)

Right now we're unable to log out and switch to another account.

When showing the ride route, enable option to also show each segment found on that route

• hr
• speed
• pace
• cadence
• altitude

Look at adding visualizations for profile stats view (goals, etc.)

http://stackoverflow.com/questions/29908345/getting-number-of-files-in-storagefolder
Use Win32 api call to get a faster count of files

We still need to expand the use of our Settings class, we'll add a feture list here as reference what's todo and what's done.

• Keep track of the last selected Activity feed filter
• Have a sorting option for Activity feed, ordered on activity start datetime or activity end datetime
• Let user decide what metric unit he prefers

Some pages have issues when rendered in landscape mode on a windows 10 phone.
For example the activity detail page, you are unable to view / scroll through the statistics in a good way.

I would rather suggest to block landscape mode, most of the layout is designed with portrait in mind.

It should be calling ConsolidateWithCache when it has a result to replace the cache entry with the actual athlete data.

Logging in as a new user should refresh the rides list.

When on mobile on the activity detail screen, the pivot will not react to swipe gestures when there is no content below the pivot header.

Example, when there are no kudos or comments given, you can't swipe from the Kudos pivot to the Comments pivot. Only way to do the swipe is by selecting the pivot header.

The current cache returns activities out of order compared to the site (but in correct timestamp order according to timezones):
App

Site

When you change the current measurement unit setting in Kliva, you'll need to restart the app to actually see the change.

We need to figure out how we can force cached data with new measurement unit.

On the original StravaDotNet git there is an open pull request:
wolfpackt99/stravadotnet@0f8dfdc

It states that the current GetClubMembers method is limited to 30 members, the pull request tries to fix this