Adding a notification system to the medical data application #25
Conversation
Added database bypass for login
Basic notifications added
InboxStyle Notification Branch
Added a updateInfoNotification & tickerTimeNotification.
Tested against API 10 to 24 and documented results
Implemented asking the user for daily updates about their personal medical information.
|
This is #9, right? 😄 |
| @@ -70,7 +71,7 @@ protected void onRestart() { | |||
| * the database last entry. | |||
| */ | |||
| private void setMainView() { | |||
| boolean firstTime = settings.getBoolean("firstTime", true); | |||
| boolean firstTime = settings.getBoolean("firstTime", false); | |||
| @@ -1,23 +1,20 @@ | |||
| package com.example.ana.exampleapp; | |||
|
|
|||
| import java.util.Calendar; | |||
There was a problem hiding this comment.
Why did you change the order of the import?
| @@ -1,37 +1,34 @@ | |||
| package com.example.ana.exampleapp; | |||
|
|
|||
| import java.lang.String; | |||
There was a problem hiding this comment.
Why did you change the order of the import?
| MongoClientURI mongoClientURI = new MongoClientURI(Variables.mongo_uri); | ||
| MongoClient mongoClient = new MongoClient(mongoClientURI); | ||
| MongoDatabase dbMongo = mongoClient.getDatabase(mongoClientURI.getDatabase()); | ||
| //MongoClientURI mongoClientURI = new MongoClientURI(Variables.mongo_uri); |
| @@ -17,14 +17,13 @@ | |||
| */ | |||
| public final class Variables { | |||
| // The server IP | |||
| private static final String IP = "IP"; | |||
| private static final String IP = "localhost"; | |||
There was a problem hiding this comment.
In ok with this change as this will be most of the time the case, but then I will add in a comment called something like Adapt the project to work in the development environment
| private static final String PORT = "27017"; | ||
| // MongoDB uri with the IP, authentication (user, password and mechanism), database, etc. | ||
| public static final String mongo_uri = "mongodb://androidUser:password@" + IP + ":" + PORT + |
There was a problem hiding this comment.
I am ok with commenting this line as this will be the case when developing, but please don't remove it as in a production environment authentication is needed. You can also added in a commit called something like Adapt the project to work in the development environment
| @@ -165,7 +175,21 @@ public boolean onOptionsItemSelected(MenuItem item) { | |||
| * @param view the {@link View} that calls the method | |||
| */ | |||
| public void btnStart(View view) { | |||
| int pin = settings.getInt("pin", 0); | |||
There was a problem hiding this comment.
why have you removed this? It seems to be for testing proposes, so please remove it!
|
I am not pretty sure if you are developing this because you plan to use it, in that case please tell me how you want to do that so I can help you. If not I will tell you what the initial idea was. 😄 |
No description provided.