Skip to content

Conversation

@msquance-stem
Copy link
Contributor

Status

Review progress:

  • Browser tested
  • Front-end review completed
  • Tech review completed

What's changed?

  • Moving custom claim capture as I had it in the wrong place

Steps to perform after deploying to production

If the production environment requires any extra work after this PR has been deployed detail it here. This could be running a Rake task, migrating a DB table, or upgrading a Gem. That kind of thing.

@sonarqubecloud
Copy link

Copy link
Contributor

@A-Wheeto A-Wheeto left a comment

Choose a reason for hiding this comment

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

LGTM

@tc-deploybot tc-deploybot temporarily deployed to teachcomputing-pr-2341 March 11, 2025 10:59 Inactive
@msquance-stem msquance-stem merged commit 1fb9b76 into main Mar 11, 2025
8 checks passed
@msquance-stem msquance-stem deleted the 3008-auth-custom-claim-capture-fix branch March 11, 2025 11:17
@sentry
Copy link

sentry bot commented Mar 11, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ **NameError: undefined local variable or method auto' for an instance of AuthController (NameError)** AuthController#callback` View Issue

Did you find this useful? React with a 👍 or 👎

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.

4 participants