Skip to content
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

Filter on account id #157

Merged
merged 21 commits into from
Nov 30, 2022
Merged

Filter on account id #157

merged 21 commits into from
Nov 30, 2022

Conversation

sunker
Copy link
Contributor

@sunker sunker commented Nov 28, 2022

This PR make it possible to filter service graph by account id and adds account id to autocomplete.

Fixes #156

sarahzinger and others added 14 commits November 8, 2022 20:03
- refactor /accountIds to /accounts to be more future proof
- make a for loop more readable
- put work behind feature flag
- move call to fetch account ids within QueryEditorForm
- fix bug to reset selection, if account id is no longer a valid option
- fix bug to handle edges with account ids with targets that do not have account ids
- remove accidentally left comments.
Filter service graph by account id and add account id to autocomplete
@sunker sunker requested a review from a team as a code owner November 28, 2022 08:17
@sunker sunker changed the title Add cross account support Filter on account id Nov 28, 2022
@CLAassistant
Copy link

CLAassistant commented Nov 28, 2022

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link

Backend code coverage report for PR #157

Plugin Main PR Difference
datasource 77.1% 75.5% -1.6%

@github-actions
Copy link

github-actions bot commented Nov 28, 2022

Frontend code coverage report for PR #157

Plugin Main PR Difference
src 88.3% 86.52% -1.78%

src/plugin.json Outdated
@@ -1,7 +1,7 @@
{
"type": "datasource",
"name": "X-Ray",
"id": "grafana-x-ray-datasource",
"id": "grafana-x-ray-datasource-private-mirror",
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ohh, thanks for finding this @kevinwcyu!

@sarahzinger sarahzinger merged commit 743a7f3 into main Nov 30, 2022
@sarahzinger sarahzinger deleted the add-cross-account-support branch November 30, 2022 14:47
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.

Make it possible to filter service graph on account id
5 participants