spontoreau / facestosmileys Goto Github PK

It could nice to allow user to save the picture on their phone

We have a problem in iOS when you need to confirm the photo to be take. Orientation is in portrait.

To deal with multiple platforms & environment

For futur demonstration, a twitter button could be nice to share pictures

Currently we define a custom ViewModel but it's better to use a framework for that. MvvmLight could be a great addition.

It can be really nice to have Toast notification to inform users on errors/informations/success ...

First case in the application is to push error notification.

As title say, project need an UWP support

Spinner stay on the main screen after camera is cancelled

Cognitive throw a BadFormat request exception when Image size are too large.

To improve the process we can:

• Take the picture in high quality.
• Scale down in a new picture to send it in Cognitive.
• Apply the Scale on Json rectangle position.

Keys are currently a problem for development.

To resolve that:

• Add a configuration class as dependencies to store keys
• Inject configuration to use keys
• When development done commit the class with empty secret keys
• Add the class to gitignore

Project need a rename !

Need a complet refactoring

Need to be generate like iOS project.

• Switch from Autofac to SimpleInjector: Lightweight & faster IoC container
• Remove ReactiveUI: Reduce package size
• Update all dependencies
• Change configuration to be Cross-platform

We just need to add a condition to check if photo was taken or not

For a better maintain we need to unify all calls to Mobile Center inside a brand new service

Dependencies are manually create so we need to had Dependency Injection

We don't need all dependencies on Cognitive service. It better for the application to request Emotion API with the HttpClient.

VSTS build step on CI return this error:

FacesToSmileys.UWP\Package.appxmanifest(21,27): Error APPX1619: App manifest references the splash screen image 'Assets\SplashScreen.scale-200.png' which does not have valid dimensions. It must be 1240x600 pixels.

@sbovo is it really an error?

The solution need to be clean to improve readability

We have an issue on Android.
When you need to take a picture the orientation is wrong in landscape mode and Cognitive didn't detect emotions