signalpoint / jDrupal

A JavaScript Library and API for Drupal Applications
http://jdrupal.tylerfrankenstein.com/
GNU General Public License v2.0
76 stars 38 forks source link

/user/login_status should be implemented before actually making attempt to login #90

Closed VishalKumarSahu closed 1 year ago

signalpoint commented 1 year ago

@VishalKumarSahu If I understand correctly, login_status was not available when I created this version of the module. Because of that, I created the connect resource in jDrupal. The idea was to call connect and that would return to you the current authentication status. From there, our apps can decide what to do.

I don't think the user login call should check login status, because I think that should be up to the developer. If login_status is now a core resource, I think we could add it as a supported resource in jDrupal. Then one could just call login_status and then make a subsequent call to login.

VishalKumarSahu commented 1 year ago

Yes it depends on developer to use login_status or not. It's a good library.