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

Fix missing output checker actions in request sidebar #57

Merged
merged 1 commit into from
Feb 13, 2019

Conversation

Jonesy
Copy link
Contributor

@Jonesy Jonesy commented Feb 13, 2019

username was added to the JWT and is used for assigning OC's to a request instead of email from before. In addition the UI uses the user.idField value to determine what to use as the username in any given deploy.

… discussions as well as the sidebar options.
@jujaga
Copy link
Member

jujaga commented Feb 13, 2019

Changes look good to me. Will be overriding CI check because this is based off of a commit prior to the blocking TF fix.

@jujaga jujaga merged commit 441efc3 into develop Feb 13, 2019
@jujaga jujaga deleted the hotfix/issue-49 branch February 13, 2019 23:08
@pripley123
Copy link
Collaborator

Tested successfully in the test environment. Resolving.

@pripley123 pripley123 added the verification Verify the bug has been fixed label Feb 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verification Verify the bug has been fixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants