Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Multi-site login #459

Open
patricksan opened this issue Oct 2, 2013 · 0 comments
Open

Multi-site login #459

patricksan opened this issue Oct 2, 2013 · 0 comments
Assignees
@httpdispatch
Labels
To be tested ★★★

Comments

@patricksan
Copy link
Member

We will start to support multiples users. So, when user log in, the login response will always return an array of credentials. If the length of the array is 1 then log the user in under that account. If it contains more than one entry then prompt the user to select which one they'd like to log into.
@ghost ghost assigned httpdispatch Oct 2, 2013
httpdispatch added a commit to httpdispatch/mobile-android that referenced this issue Oct 2, 2013
@httpdispatch
Issue photo#459 (Card 7) (Multi-site login)
11447ec 
app:
- AccountLogin.LogInFragment, AccountSignup.NewUserFragment: now it
implements LoginActionHandler
- AccountLogin.LogInFragment, AccountSignup.NewUserFragment: added
static fields sCurrentInstance and sCurrentInstanceAccessor
- AccountLogin.LogInFragment, AccountSignup.NewUserFragment: added new
fields mDelayedLoginProcess, mLastResponse
- AccountLogin.LogInFragment, AccountSignup.NewUserFragment: added
implementation of onCreate and onDestroy to init sCurrentInstance static
field
- AccountLogin.LogInFragment, AccountSignup.NewUserFragment,
GoogleLoginFragment: added new methods processLoginCredentials,
processLoginResponse, onViewCreated
- AccountLogin.LogInFragment.LogInUserTask,
AccountSignup.NewUserFragment.NewUserTask,
GoogleLoginFragment.LogInUserTask: reworked
onSuccessPostExecuteAdditional method to call processLoginResonse or
schedule delayed login processing depend on fragment-activity attached
state
- GoogleLoginFragment: renamed currentInstance field to
sCurrentInstance, requestCode to mRequestCode, delayedLoggedIn to
mDelayedLoginProcessing
- GoogleLoginFragment: added new static field sCurrentInstanceAccessor
- GoogleLoginFragment: added new field mLastResponse
- GoogleLoginFragment: removed method onLoggedIn and reworked
onViewCreated to call processLoginResonse
- SetupActivity: replaced getActivity with getSupportActivity call for
OAuthUtils.askOAuth call
- Credentials: added
- AccountTroveboxResponse: removed different credentials field and added
mCredentials field
- AccountTroveboxResponse: reworked constructor to support array of
credentials in the json result
- OAuthUtils: replaced Activity reference in import with HE one
- LoginUtils: added new methods: onLoggedIn,
processSuccessfulLoginResult, performLogin
- LoginUtils.SelectAccountSelectedActionHandler: added
- LoginUtils.LoginActionHandler: added
- LoginUtils.SelectAccountDialogFragment: added
- res/values/strings.xml, res/values-ru/strings.xml: added
select_trovebox_account constant

test:
- CredentialsTest: added
- AccountTroveboxApiTest: modified to use credentials reference from the
modified AccountTroveboxResponse
- AccountTroveboxResponseTest: added
- res/raw/json_credentials.txt: added
- res/raw/json_login_multiple.txt: added
- res/raw/json_login_simple.txt: added
httpdispatch added a commit to httpdispatch/mobile-android that referenced this issue Oct 10, 2013
@httpdispatch
Issue photo#459 (Card 7) (Multi-site login)
c35fa49 
app:
- Credentials: renamed mServer field to mHost. Added getHost method and
changed internals of getServer method.
- Credentials: removed setters for fields
- LoginUtils.SelectAccountDialogFragment: replaced emails with host in
the select account dialog
test:
- CredentialsTest: added checkCredentialsV2 method
- AccountTroveboxResponseTest: added testMultiResponseV2 test
- res/raw/json_login_multiple_v2.txt: added
httpdispatch added a commit to httpdispatch/mobile-android that referenced this issue Oct 17, 2013
@httpdispatch
Issue photo#459 (Card 7) (Multi-site login)
8689e71 
app:
- AccountLogin, AccountSignup: added registering of destroy on login
broadcast receiver to the onCreate method
- AccountLogin.LogInFragment, AccountSignup.NewUserFragment: modified
parameter to do not self close activity for the LoginUtils.onLoggedIn
call in the processLoginCredentials method
- GalleryFragment.CustomImageFetcher: updated refrence to imageHeight in
processBitmap method
- SelectAccountActivity: added
- ImageFetcher: added mCheckLoggedIn field and its setter and getter
- ImageFetche, ImageFileSystemFetcherr: updated references to imageWidth
and imageHeight fields in the processBitmap method
- ImageFetcher: added use of mCheckLoggedIn field parameter to the
downloadBitmap method call in the processBitmap method
- ImageFetcher: added additional checkLoggedIn parameter to the
downloadBitmap method
- ImageResizer: updated imageWidth and imageHeight field names
- Credentials: added mProfileInformation field and its support to
constructor and parcelable implementation
- LoginUtils: modified processSuccessfulLoginResult to show
SelectAccountActivity in case of  multiple login credentials instead of
dialog
- LoginUtils.SelectAccountSelectedActionHandler: removed
- LoginUtils.SelectAccountDialogFragment: removed
- SimpleViewLoadingControl: added
- res/layout/fragment_select_account.xml: added
- res/layout/list_item_account.xml: added
- res/values/dimens.xml: added new constants for select account window
thumbs
- res/values/strings.xml, res/values-ru/strings.xml: added new constants
for multiple upload notification and updated select_trovebox_account
constant
- AndroidManifest.xml: added SelectAccountActivity declaration
- AndroidManifest.xml: updated version code and name

test:
- CredentialsTest: added testing of newly added profileInformation field
to Credentials to the testCredentialsParcelableV2 method
- res/raw/json_credentials_v2.txt: fixed some data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@httpdispatch httpdispatch

Labels
To be tested ★★★
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@patricksan @httpdispatch