Skip to content

Permission Items: Common code #1471

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

Merged
merged 3 commits into from
Apr 10, 2024

Conversation

julienduchesne
Copy link
Contributor

@julienduchesne julienduchesne commented Apr 8, 2024

Follow-up to #1465 and #1470
Lots of the code is exactly the same since it's the same API. This PR puts it in a common struct so that other permissions resources will be easier to add

Some of this code will also be usable for the global permissions resources (the existing not-_item ones)

Copy link

github-actions bot commented Apr 8, 2024

In order to lower resource usage and have a faster runtime, PRs will not run Cloud tests automatically.
To do so, a Grafana Labs employee must trigger the cloud acceptance tests workflow manually.

@julienduchesne julienduchesne force-pushed the julienduchesne/permission-item-common-code branch from 38a76ba to 7435d9d Compare April 8, 2024 17:36
@julienduchesne julienduchesne marked this pull request as ready for review April 8, 2024 20:12
@julienduchesne julienduchesne requested a review from a team as a code owner April 8, 2024 20:12
@julienduchesne julienduchesne force-pushed the julienduchesne/ds-permission-item branch from 39e09a2 to 2cbc58c Compare April 10, 2024 12:35
Base automatically changed from julienduchesne/ds-permission-item to main April 10, 2024 13:28
Follow-up to #1465 and #1470
Lots of the code is exactly the same since it's the same API. This PR puts it in a common struct so that other permissions resources will be easier to add
@julienduchesne julienduchesne force-pushed the julienduchesne/permission-item-common-code branch from b5aec83 to df51981 Compare April 10, 2024 13:29
@julienduchesne julienduchesne merged commit 549e3e2 into main Apr 10, 2024
@julienduchesne julienduchesne deleted the julienduchesne/permission-item-common-code branch April 10, 2024 13:38
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