-
Notifications
You must be signed in to change notification settings - Fork 970
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
Logging of interesting events #201
Comments
Welcome @jdormansteele Thanks for a good tip. We'd like to see the events getting logged to a file as well. Perhaps a new feature request? (I'll reassign this.) |
@jdormansteele welcome to our project! Your opened Issue is actually something I've been thinking about for quite some time now. @E3V3A, should we move the Main Screen aside so that it is still accessible via the Navigation Drawer and add the recent events as the screen that the user sees first? |
Having a list of recent events with coordinates where each was observed and a link to open those in a map would be my ideal first screen upon opening the app. I've seen alerts while my phone was on silent and had no idea when they'd occurred. |
@leecolleton, how awesome to see you back here on GitHub! Thing is, we are still in ALPHA stage and despite having a large follower base currently lack developers to erase the current bugs. If you would like to force this to be implemented, please don't hesitate to craft a pull request for this. It is a sad thing that we have 70+ forks, but close to no pull requests at all. I don't really get why people won't contribute. @jdormansteele, I am just throwing in two screenshots of other programs out there. We will likely have a totally different design for this, but we are always open for any improvement suggestions. Which general style do you prefer? Are there any specific things you would definitely like to see in this screen? |
@SecUpwN: Both presentations would be good together -- a graph on the front-end, clickable to open a detailed, editable list of events. The colors on the graph seem right except for the white, which to me suggests "missing information" rather than "no problem." And only "no suspicion" should be green. |
We have a new view called "debug log" implemented. I want to use this as a quick solution to see what the program is doing. So the question, how to show a positive detection on the Screen can be discussed and developed later. The only thing we have to do is print out the values to the logcat. for example:
I keep working on that. If anybody has another ideas, let me know. |
Thanks for your feedback, @jdormansteele. How about if we just color the bars in the colors of our Status Icons? Which brings me to another question for @E3V3A and @He3556: Can you please review the Detailed Explanations on the Status Icons page? Would you please expand those with correct explanations? I want this to reflect the current usage of our Icons so that users know what they shall do. |
@SecUpwN These review requests really deserve their own issues. Please make one and I'll add my thoughts there. We need to re-consider the words there... |
@He3556 YES, this is definitely something we would like to have. As I see it there are really only two options to show and save event logs.
We then have to visualize these events. Perhaps either or both:
[ From the RF Signal Tracker app.] |
@d4rken, you where the first person I had in mind for this Issue since you coded the awesome trust event logger. Would you please read this whole Issue and see if you can create a pull request for this, changing the main view to what is proposed here? |
ok i start with a. the simple table :) i will update my fork this weekend. Can't wait to do it - still have some work to do before my holliday starts. |
@SecUpwN It's just another logger with a 1 MB of eyecandy! I really don't think we need that. Built-in SQLIte3 can pop out anything in CSV,TSV and several other formats with one little command, and to a file of your choice... We can even use the already built in DB viewer to do this. So let's not ask people to commit huge PR's to stuff we don't need and have no idea what else it does... |
@d4rken Yes, thanks, no problem. |
I've added a new table called |
Ok so I see why this is a hard issue. We (devs) haven't at all specified _what_ to put in this
In more detail:
Any other suggestions? |
If we have implemented all the things you've just mentioned, we're close to BETA and people will tell. |
Thanks for the useful and timely app.
I wonder whether there is a logging of alerts. It seems that the cell/location information saved in the database doesn't actually have anything to do with events that would generate alerts.
Keep up the good work!
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
The text was updated successfully, but these errors were encountered: