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.

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

Add AuthZ to Agents, Ratings and Schedules #1489

Merged
merged 1 commit into from
May 28, 2024
Merged

Add AuthZ to Agents, Ratings and Schedules #1489

merged 1 commit into from
May 28, 2024

Conversation

tylerwowen
Copy link
Contributor

@tylerwowen tylerwowen commented Mar 2, 2024 •
edited
Loading

These 3 resurrects previously didn't have Authorization. Added in this separate PR since they are breaking change.

Stack:

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

@tylerwowen tylerwowen requested a review from a team as a code owner March 2, 2024 00:26
@github-actions github-actions bot added the deploy-service Includes changes to deploy-service label Mar 2, 2024
@tylerwowen tylerwowen removed the request for review from a team March 2, 2024 00:27
@tylerwowen tylerwowen marked this pull request as draft March 2, 2024 00:27
@tylerwowen tylerwowen added this to the Teletraan AuthN & AuthZ milestone Mar 2, 2024
@tylerwowen tylerwowen mentioned this pull request Mar 4, 2024
Merged
@tylerwowen tylerwowen force-pushed the spr/master/aa055770 branch 2 times, most recently from a0f3d97 to dc7f06f Compare March 4, 2024 22:20
@tylerwowen tylerwowen force-pushed the spr/master/eb5c1322 branch 2 times, most recently from b841730 to fefc340 Compare March 6, 2024 01:37
@github-actions github-actions bot added the deploy-board Includes changes to deploy-board label Mar 12, 2024
@tylerwowen tylerwowen force-pushed the spr/master/aa055770 branch 2 times, most recently from 22b9901 to 3a1fcba Compare March 28, 2024 00:56
Base automatically changed from spr/master/eb5c1322 to master April 3, 2024 00:46
@tylerwowen tylerwowen dismissed ntascii’s stale review April 3, 2024 00:46

The base branch was changed.

@tylerwowen tylerwowen mentioned this pull request May 2, 2024
Merged
@tylerwowen tylerwowen changed the base branch from master to spr/master/ef160c01 May 15, 2024 00:28
@tylerwowen tylerwowen requested review from a team and yongwen May 15, 2024 00:30
@tylerwowen tylerwowen force-pushed the spr/master/ef160c01 branch 2 times, most recently from ee4f20a to 3dead2a Compare May 15, 2024 22:11
@tylerwowen tylerwowen force-pushed the spr/master/aa055770 branch 2 times, most recently from 088dea6 to cc70956 Compare May 15, 2024 22:39
Base automatically changed from spr/master/ef160c01 to master May 25, 2024 00:01
@tylerwowen tylerwowen requested a review from osoriano May 25, 2024 00:02
@tylerwowen tylerwowen merged commit abd5c03 into master May 28, 2024
5 checks passed
@tylerwowen tylerwowen deleted the spr/master/aa055770 branch May 28, 2024 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@osoriano osoriano osoriano approved these changes

@ntascii ntascii ntascii left review comments

@yongwen yongwen Awaiting requested review from yongwen

Assignees
No one assigned
Labels
deploy-service Includes changes to deploy-service
Projects
None yet
Milestone
Teletraan AuthN & AuthZ
Development

Successfully merging this pull request may close these issues.
3 participants
@tylerwowen @osoriano @ntascii