-
Notifications
You must be signed in to change notification settings - Fork 481
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.
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
Change SimpleSubmitHandler and angular page to use new DataTypes from portability-types-client #107
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor comments
export class SimpleLoginComponent implements OnInit { | ||
url = environment.apiPostUrl; | ||
username: string = ""; | ||
password: string = ""; | ||
enableSubmit: boolean = false; | ||
error_text: string = ""; | ||
constructor() { } | ||
constructor(private service : BackendService) { } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Run ng test and e2e tests to make sure we don't have to update anything to make them pass. Travis should pick up the failure, if there is one, too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
@ApiModel | ||
public class SimpleLoginRequest { | ||
private final String username; | ||
private final String password; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: Determine how to avoid storing the password in the clear.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. Note that this is also only used in the microsoft mail auth flow. We can delete it once we move that over to oauth or Live Sync.
This requires both a binary push and a static content deploy (#67)
This is the last piece to switching the portability-api to use the new data types (#71)