CMPUT301W13T06 / classproject

RecipeBot
http://cmput301w13t06.github.com/classproject/
2 stars 6 forks source link

Add ElasticSearch #49

Closed f2prateek closed 11 years ago

f2prateek commented 11 years ago

@btrinh1 , @saturna , @ethanmykytiuk Can you guys think of any other operations we should setup for the backend?

Right Now the operations are -

  1. Insert
  2. Delete
  3. Update (could be improved)
  4. Search By Name
  5. Search by Ingredients (either all given ingredients, or any of the given ingredients).
gptrinh commented 11 years ago

Hmm nothing comes to mind at this moment. That should cover most of the use cases. I've been looking at the new recipe activity that you did and am not sure if you intend on adding anything else or fixing things. I'm assuming the listviews are supposed to be larger than a single line but do you want me to add the buttons for adding and such now or do you intend to do so later? If anything I'll be looking up how the camera intent works and trying to incorporate a alert dialog for adding additional images.

On Wed, Mar 20, 2013 at 11:46 PM, Prateek Srivastava < notifications@github.com> wrote:

@btrinh1 https://github.com/btrinh1 , @saturnahttps://github.com/saturna, @ethanmykytiuk https://github.com/ethanmykytiuk Can you guys think of any other operations we should setup for the backend?

Right Now the operations are -

  1. Insert
  2. Delete
  3. Update (could be improved)
  4. Search By Name
  5. Search by Ingredients (either all given ingredients, or any of the given ingredients).

— Reply to this email directly or view it on GitHubhttps://github.com/CMPUT301W13T06/classproject/pull/49#issuecomment-15220619 .

f2prateek commented 11 years ago

For the Recipe Activity I'm thinking of just re-arranging the layout for now.

gptrinh commented 11 years ago

Okay then I'll just be working on the dialog and such for adding images for now

On Thu, Mar 21, 2013 at 12:41 AM, Prateek Srivastava < notifications@github.com> wrote:

For the Recipe Activity I'm thinking of just re-arranging the layout for now.

— Reply to this email directly or view it on GitHubhttps://github.com/CMPUT301W13T06/classproject/pull/49#issuecomment-15221806 .

f2prateek commented 11 years ago

I think that's already working, only the ability to take it form the camera directly isn't working. You should be able to re-use most of the code form the PICK_IMAGE intent, in the onActivtyResult, only thing that will change is the launch intent.

gptrinh commented 11 years ago

Oh woops yeah I meant the directly from camera feature. I'm not sure I'm following you I was going to create an AlertDialog in the View.onClickListener and have two buttons one would essentially load image from gallery as it is now and the other would allow for direct camera load via: Intent intent = new Intent(MediaStore.ACTION_IMAGE_CAPTU RE); then I would just pass the new intent to startActivityResult(intent, CAPTURE_IMAGE_ACTIVITY_REQUEST_C ODE); Would that work? or is the onActivityResult not be resusable?

On Thu, Mar 21, 2013 at 12:45 AM, Prateek Srivastava < notifications@github.com> wrote:

I think that's already working, only the ability to take it form the camera directly isn't working. You should be able to re-use most of the code form the PICK_IMAGE intent, in the onActivtyResult, only thing that will change is the launch intent.

— Reply to this email directly or view it on GitHubhttps://github.com/CMPUT301W13T06/classproject/pull/49#issuecomment-15221885 .

f2prateek commented 11 years ago

Instead of having a dialog, it will be better to have an icon in the menu bar that would launch the camera activity. Dialogs should be used sparingly. I can explain in detail in the lab tomorrow. On Mar 21, 2013 1:02 AM, "btrinh1" notifications@github.com wrote:

Oh woops yeah I meant the directly from camera feature. I'm not sure I'm following you I was going to create an AlertDialog in the View.onClickListener and have two buttons one would essentially load image from gallery as it is now and the other would allow for direct camera load via: Intent intent = new Intent(MediaStore.ACTION_IMAGE_CAPTU RE); then I would just pass the new intent to startActivityResult(intent, CAPTURE_IMAGE_ACTIVITY_REQUEST_C ODE); Would that work? or is the onActivityResult not be resusable?

On Thu, Mar 21, 2013 at 12:45 AM, Prateek Srivastava < notifications@github.com> wrote:

I think that's already working, only the ability to take it form the camera directly isn't working. You should be able to re-use most of the code form the PICK_IMAGE intent, in the onActivtyResult, only thing that will change is the launch intent.

— Reply to this email directly or view it on GitHub< https://github.com/CMPUT301W13T06/classproject/pull/49#issuecomment-15221885>

.

— Reply to this email directly or view it on GitHubhttps://github.com/CMPUT301W13T06/classproject/pull/49#issuecomment-15222295 .

gptrinh commented 11 years ago

Okay sounds good I'll just see you tomorrow then.

On Thu, Mar 21, 2013 at 1:05 AM, Prateek Srivastava < notifications@github.com> wrote:

Instead of having a dialog, it will be better to have an icon in the menu bar that would launch the camera activity. Dialogs should be used sparingly. I can explain in detail in the lab tomorrow. On Mar 21, 2013 1:02 AM, "btrinh1" notifications@github.com wrote:

Oh woops yeah I meant the directly from camera feature. I'm not sure I'm following you I was going to create an AlertDialog in the View.onClickListener and have two buttons one would essentially load image from gallery as it is now and the other would allow for direct camera load via: Intent intent = new Intent(MediaStore.ACTION_IMAGE_CAPTU RE); then I would just pass the new intent to startActivityResult(intent, CAPTURE_IMAGE_ACTIVITY_REQUEST_C ODE); Would that work? or is the onActivityResult not be resusable?

On Thu, Mar 21, 2013 at 12:45 AM, Prateek Srivastava < notifications@github.com> wrote:

I think that's already working, only the ability to take it form the camera directly isn't working. You should be able to re-use most of the code form the PICK_IMAGE intent, in the onActivtyResult, only thing that will change is the launch intent.

— Reply to this email directly or view it on GitHub<

https://github.com/CMPUT301W13T06/classproject/pull/49#issuecomment-15221885>

.

— Reply to this email directly or view it on GitHub< https://github.com/CMPUT301W13T06/classproject/pull/49#issuecomment-15222295>

.

— Reply to this email directly or view it on GitHubhttps://github.com/CMPUT301W13T06/classproject/pull/49#issuecomment-15222360 .