search

e-plant-se / Second_try

This is a platform that is meant to be used as a tool for registration and control for your urban garden.
0 stars 0 forks source link

Non-showed message #7

Open Colima-97 opened 5 years ago

Colima-97 commented 5 years ago

@PhonyFish, in the view AgregarCamaPage.xaml.cs, when you click the button Guardar and no information is given, there is no message showed.

private async void Button_Clicked(object sender, EventArgs e) { await _vm.PostCamaAsync(huerto.Text, idCama.Text, tipo.Text, estructura.Text, planta.Text, fechaCultivo.Text); }

At least, Firebase helps you out with it, if no data is given, it won't save anything. But you must show something.

Also, why do you have a SelectPhoto method? You don't even need it.

private async void SelectPhoto(object sender, System.EventArgs e) { if (CrossMedia.Current.IsTakePhotoSupported) { var imagen = await CrossMedia.Current.PickPhotoAsync(); var stream = imagen.GetStream(); streamFile = stream; if (imagen != null) { var guid = Guid.NewGuid(); image.IsVisible = true; streamImage = guid.ToString(); image.Source = ImageSource.FromStream(imagen.GetStream); } } }

luis-ortiz-dev commented 5 years ago

oks, I'll see what I can do about that message. About the image method, beats me why it's there, I told carlitos it wasn't needed

Colima-97 commented 5 years ago

Ok buddy, see what you can do. And upload the changes on master.