instedd / alto_guisso Goto Github PK

RestClient used by Guisso::Api::BasicAuth and the httpclient of Guisso::Api::Oauth does not obey the same interface.

Found in version fc884822567c8a7f44cebf49894fd8e485cbfa77

1. Go to Resource Map (version 2.14-pre2, also happens with mBuilder v1.1-63-g6224c7d and Verboice v2.3.7-4-gc92444b)
2. Click on 'Log in'
3. Click 'Sign in with Google' or sign in with email and password
4. After being redirected to Resource Map, click on the user's email and in the drop-down menu click 'Sign out'
5. Click 'Log in' again

Expected: when you sign out a notice that reads "Signed out successfully" appears in the Resource Map app and when you click 'Log in' for the second time you get redirected to the InSTEDD Platform.

Obtained: when you click 'Log in' for the second time you get redirected to the InSTEDD Platform and another "Signed out successfully" notice appears.

Version fc884822567c8a7f44cebf49894fd8e485cbfa77

To differentiate the 'Settings' page in the InSTEDD tools with the one specifically on Guisso, it would be useful to add a 'My Account' page where the user could see the information or make changes referring the InSTEDD account (for example: Change password).

As the user, the link 'My Account' should be found inside the drop-down along with the links for 'Settings', 'Report a bug' and 'Sign out'.

The design for the page is in the works by @jkicillof

It would be more coherent if the "Sign in" and "Sign out" links in Guisso were replaced with the "Log in" and "Log out" links showed in all the InSTEDD tools.
Also Replace "Sign up" with "Create account"

Found in version fc884822567c8a7f44cebf49894fd8e485cbfa77

Currently, the Sign up page only has the option to create an account completing the Email and Password inputs and it would be useful to also add the "Sign in with Google" button that appears in the 'Sing in' page.

In version fc884822567c8a7f44cebf49894fd8e485cbfa77

It is somewhat confusing to understand when to type the new password:

It would be better if the inputs were rearranged: first you type your current password and then the new one and its confirmation. Instead of 'Password' it should said 'New password' and 'New password confirmation' in the input right under.

Also, considering that there is a 'Back' link and that adding the current password is not a requirement to leave the Settings page, it would helpful to delete the text "leave blank if you don't want to change it" and add a 'Cancel' link next to the 'Update' button and a disclaimer so that it is even more clear that the password is going to change for all InSTEDD tools.

RestClient is a dependency of BasicAuth it should be listed in .gemspec file