Skip to content

[NEW][APPS] Allow apps to modify a subset of global settings #25913

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.

Sign up for GitHub

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

Jump to bottom
Merged
kodiakhq merged 10 commits into from
Jun 29, 2022
Merged

[NEW][APPS] Allow apps to modify a subset of global settings #25913

kodiakhq merged 10 commits into from
Jun 29, 2022

Conversation

@pierre-lehnen-rc
Copy link
Contributor

@pierre-lehnen-rc pierre-lehnen-rc commented Jun 17, 2022

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@pierre-lehnen-rc pierre-lehnen-rc requested a review from a team as a code owner June 17, 2022 18:33
@github-actions github-actions bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Jun 18, 2022
@pierre-lehnen-rc pierre-lehnen-rc added this to the 5.0.0 milestone Jun 19, 2022
@pierre-lehnen-rc pierre-lehnen-rc mentioned this pull request Jun 19, 2022
Merged
@kodiakhq kodiakhq bot removed the stat: ready to merge PR tested and approved waiting for merge label Jun 22, 2022
@kodiakhq
Copy link
Contributor

kodiakhq bot commented Jun 22, 2022

This PR currently has a merge conflict. Please resolve this and then re-add the ['stat: ready to merge', 'automerge'] label.

@ggazzo ggazzo added the stat: ready to merge PR tested and approved waiting for merge label Jun 27, 2022
@pierre-lehnen-rc pierre-lehnen-rc modified the milestone: 5.0.0 Jun 28, 2022
@d-gubert d-gubert removed the stat: ready to merge PR tested and approved waiting for merge label Jun 28, 2022
@ggazzo ggazzo added the stat: ready to merge PR tested and approved waiting for merge label Jun 28, 2022
@ggazzo ggazzo added stat: needs QA and removed stat: ready to merge PR tested and approved waiting for merge labels Jun 28, 2022
@d-gubert d-gubert changed the title [NEW] Allow apps to modify a subset of global settings [NEW][APPS] Allow apps to modify a subset of global settings Jun 28, 2022
@ggazzo ggazzo added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Jun 28, 2022
@kodiakhq kodiakhq bot merged commit a6ef22a into develop Jun 29, 2022
@kodiakhq kodiakhq bot deleted the new/apps-update-settings branch June 29, 2022 13:13
@murtaza98 murtaza98 mentioned this pull request Jul 21, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@d-gubert d-gubert d-gubert approved these changes

Assignees

No one assigned

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Milestone

5.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@pierre-lehnen-rc @d-gubert @ggazzo