Skip to content

Pass.muxi tech.xyz auth fix#28

Merged
zxc0328 merged 3 commits intorefactor-passfrom
pass.muxi-tech.xyz_auth_fix
Jul 9, 2021
Merged

Pass.muxi tech.xyz auth fix#28
zxc0328 merged 3 commits intorefactor-passfrom
pass.muxi-tech.xyz_auth_fix

Conversation

@qingxuanying
Copy link
Copy Markdown
Contributor

No description provided.

@@ -0,0 +1,158 @@
import React, { Component } from 'react';
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

login 文件夹是老的登陆,login_auth 是新的吗,如果是这样,你在路由里面要给这个组件加一个新的路由啊。不然用户怎么访问。

Comment thread src/pages/login_auth/login.js Outdated
async login() {
const { username, password, isChecked } = this.state;
if (username && password) {
const res = await Service.getOauthCode(username, password);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

client_id 写死了?

Comment thread src/pages/login/login.js
).then(res => {
console.log(res);
if (res !== null && res !== undefined) {
this.setState({
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

这个文件确定恢复到之前的状态了吧?如果不确定可以直接去 Git 历史里找老的文件,复制粘贴

Copy link
Copy Markdown
Member

@zxc0328 zxc0328 left a comment

Choose a reason for hiding this comment

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

LGTM

@zxc0328 zxc0328 merged commit 848c4a6 into refactor-pass Jul 9, 2021
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.

2 participants