Skip to content

Register user#44

Merged
ggazzo merged 22 commits intodevelopfrom
register-user
Nov 7, 2017
Merged

Register user#44
ggazzo merged 22 commits intodevelopfrom
register-user

Conversation

@gilmarsquinelato
Copy link
Contributor

Created a component with fields Name, Email and password.
Created actions to receive the user data and dispatch to rocketchat lib to create user.

Copy link
Member

@ggazzo ggazzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gilmarsquinelato nice code! but I think we have some problems :x.

  • we are creating users without usernames(you can create one more screen asking for username, like the web client or use the REST API that accept username as param)
  • if the creation returns ok we are redirect to login, you can get the token access by the success of method and redirect to home.

@gilmarsquinelato
Copy link
Contributor Author

@ggazzo Thanks for the feedback, I'll make the changes :)

@codecov-io
Copy link

Codecov Report

Merging #44 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           develop    #44   +/-   ##
======================================
  Coverage      100%   100%           
======================================
  Files            6      6           
  Lines           44     44           
  Branches         9      9           
======================================
  Hits            44     44

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0777a9...470b83d. Read the comment docs.

@codecov-io
Copy link

codecov-io commented Sep 27, 2017

Codecov Report

Merging #44 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           develop    #44   +/-   ##
======================================
  Coverage      100%   100%           
======================================
  Files            6      6           
  Lines           44     44           
  Branches         9      9           
======================================
  Hits            44     44

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 335f299...464bcba. Read the comment docs.

yield takeEvery(types.LOGIN.REQUEST, handleLoginRequest);
yield takeEvery(types.LOGIN.SUCCESS, saveToken);
yield takeEvery(types.LOGIN.SUBMIT, handleLoginSubmit);
yield takeEvery(types.LOGIN.REGISTER_REQUEST, handleRegisterRequest);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use takelatest

};
const root = function* root() {
yield takeEvery(types.LOGIN.SUCCESS, watchRoomsRequest);
yield takeEvery(types.ROOMS.REQUEST, watchRoomsRequest);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should call roomsRequest in some place :)

ggazzo
ggazzo previously approved these changes Nov 7, 2017
@ggazzo ggazzo deleted the register-user branch November 7, 2017 16:28
@ggazzo ggazzo mentioned this pull request Nov 7, 2017
djorkaeffalexandre pushed a commit that referenced this pull request Aug 9, 2019
* Added feature to register a new user

* login after register working

* Removed username from register and placed on a new view

* loading indicator on username submit

* register/username logo layout issue

* - login and register background white

* - logo removed from logo and register
@Lao-Ax Lao-Ax mentioned this pull request Dec 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants