Skip to content

[TE] frontend - harleyjj/share-dashboard - extend anomalies by subscr…#4132

Merged
jihaozh merged 1 commit intoapache:masterfrom
harleyjj:share-dashboard
Apr 18, 2019
Merged

[TE] frontend - harleyjj/share-dashboard - extend anomalies by subscr…#4132
jihaozh merged 1 commit intoapache:masterfrom
harleyjj:share-dashboard

Conversation

@harleyjj
Copy link
Contributor

…iption group to share-dashboard

The user can share anomalies in an email, link, or pdf grouped by subscription group, as an alternative to the present sharing functionality of anomalies by application.

} else if (this.get('appName')) {
anomalies = yield this.get('anomaliesApiService').queryAnomaliesByAppName(this.get('appName'), this.get('startDate'), this.get('endDate'));
} else {
anomalies = yield this.get('anomaliesApiService').queryAnomaliesBySubGroup(this.get('subGroup'), this.get('startDate'), this.get('endDate'));
Copy link
Member

Choose a reason for hiding this comment

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

Which endpoint is this calling? So that I can double check the backend to see if it's correct.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Here's an example, with appName and group seeming to have join behavior, based on the responses I've gotten:

/userdashboard/anomalies?appName={someAppName}group={someSubGroup}

Copy link
Member

Choose a reason for hiding this comment

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

Right. Doubled check this will work.

Copy link
Member

@jihaozh jihaozh left a comment

Choose a reason for hiding this comment

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

Awesome!

@jihaozh jihaozh merged commit c18a785 into apache:master Apr 18, 2019
@harleyjj harleyjj deleted the share-dashboard branch April 18, 2019 23:04
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