Skip to content

Fix admin reportform dateformat error#26

Merged
wilhelmguo merged 1 commit intoQihoo360:masterfrom
wilhelmguo:hotfix/fix_admin_reportform_dateformat
Nov 20, 2018
Merged

Fix admin reportform dateformat error#26
wilhelmguo merged 1 commit intoQihoo360:masterfrom
wilhelmguo:hotfix/fix_admin_reportform_dateformat

Conversation

@wilhelmguo
Copy link
Copy Markdown
Collaborator

No description provided.

@wilhelmguo wilhelmguo added sig/frontend Categorizes an issue or PR as relevant to sig-frontend. kind/bug Categorizes issue or PR as related to a bug. labels Nov 20, 2018
@BennieMeng
Copy link
Copy Markdown
Collaborator

/lgtm

@wilhelmguo wilhelmguo merged commit 4b3034d into Qihoo360:master Nov 20, 2018

refresh(state?: State) {
this.publishService.getDeployStatistics(this.startTime + 'T00:00:00Z', this.endTime + 'T00:00:00Z').subscribe(
this.publishService.getDeployStatistics(moment(this.startTime).format("YYYY-MM-DDTHH:mm:SS")+'Z', moment(this.endTime).format("YYYY-MM-DDTHH:mm:SS")+'Z').subscribe(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

这种最好也提一个common func吧,影响不大

@wilhelmguo
Copy link
Copy Markdown
Collaborator Author

wilhelmguo commented Nov 20, 2018

#20
#13

@wilhelmguo wilhelmguo changed the title frontend: fix admin reportform dateformat error Fix admin reportform dateformat error Dec 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Categorizes issue or PR as related to a bug. sig/frontend Categorizes an issue or PR as relevant to sig-frontend.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants