Skip to content

fix: Modify overview query for different role#438

Merged
sunnywx merged 1 commit into
openpitrix:masterfrom
whDongRui:overview/top
Sep 30, 2018
Merged

fix: Modify overview query for different role#438
sunnywx merged 1 commit into
openpitrix:masterfrom
whDongRui:overview/top

Conversation

@whDongRui

Copy link
Copy Markdown
Contributor
  • fix: Modify meun styles

@whDongRui
whDongRui requested a review from sunnywx September 29, 2018 12:12
@codecov

codecov Bot commented Sep 29, 2018

Copy link
Copy Markdown

Codecov Report

Merging #438 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #438   +/-   ##
=======================================
  Coverage   17.69%   17.69%           
=======================================
  Files          86       86           
  Lines        1373     1373           
  Branches      286      286           
=======================================
  Hits          243      243           
  Misses       1075     1075           
  Partials       55       55
Impacted Files Coverage Δ
src/utils/index.js 1.01% <ø> (ø) ⬆️

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 b5c2f7b...9c41755. Read the comment docs.

Comment thread src/stores/UserStore.js
isDev: userInfo.role === 'developer',
isNormal: userInfo.role === 'user',
loginTime: new Date()
loginTime: Date.parse(new Date())

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Date.now() or +new Date

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok. Modify Date.now().

@@ -20,46 +22,43 @@ export default class RepoList extends PureComponent {

static defaultProps = {
type: 'public',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

default type not in range of prop-types

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok, default "repo"

@sunnywx
sunnywx merged commit 2e2f175 into openpitrix:master Sep 30, 2018
@whDongRui
whDongRui deleted the overview/top branch January 17, 2019 06:52
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