Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display pending updates in the status messages #371

Open
oliv3 opened this issue Oct 3, 2018 · 5 comments
Open

Display pending updates in the status messages #371

oliv3 opened this issue Oct 3, 2018 · 5 comments

Comments

@oliv3
Copy link
Contributor

oliv3 commented Oct 3, 2018

This adds another functional test that is to display the number of pending updates.

screenshot_20181003-144538
Unsent positions count will appear in parenthesises.
Another way for the user to check for e.g. network issue.

oliv3 added a commit to oliv3/traccar-client-android that referenced this issue Oct 3, 2018
@redge76
Copy link
Contributor

redge76 commented Nov 28, 2018

I also interested by some statistical data but I don't think that adding only the number of pending locations to the "update" message is the right way.
I think you could use another "view" or keep the data in a "text box" at the top of the status page.
Another solution is to open a "Dialog" when you touch one of the line. It's a good solution to display a lot of information.

Stats I'd like to see are:

  • number of pending locations
  • time since last update
  • current GPS status & accuracy

Also, it is not obvious that the number in "( )" is the number of pending location.

@oliv3
Copy link
Contributor Author

oliv3 commented Nov 28, 2018

Feel free to send a PR. Can't spend more time on this, my customers are fine with the feature as it is.

@redge76
Copy link
Contributor

redge76 commented Nov 28, 2018

ohhh it was also not obvious it was a dev for some customer.
My message was just a "user feedback" and I don't have much time too.... That's the way opensource go. Hard to find contributors... Sorry.

@oliv3
Copy link
Contributor Author

oliv3 commented Nov 28, 2018

It is fine to be integrated, as is. This PR is to off-load work on my side :)
Could surely be better to add some message for the () meaning, but when properly documented or explained, people understand.
And I don't think @tananaev does want to make the application complicated, (not to say user-friendly), so I decided to keep this bare bones.

@oliv3
Copy link
Contributor Author

oliv3 commented Nov 29, 2018

@redge76 also, this feature would have to be coded for iOS, for it to be merged by @tananaev :(

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

No branches or pull requests

2 participants