-
Notifications
You must be signed in to change notification settings - Fork 3k
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
revert conditional in auth/command/TakePolicyOwnership.cpp
#47528
Comments
Not overdue, held on https://github.com/Expensify/Auth/pull/11946 |
@blimpich Whoops! This issue is 2 days overdue. Let's get this updated quick! |
still waiting on auth PR to get merged and then the web pr |
Not overdue, status unchanged, still held |
Not overdue, PR is out, waiting for https://github.com/Expensify/Web-Expensify/pull/43074 to deploy to production, which is when we'll be able to get a green build on the PR for this issue. |
@blimpich Huh... This is 4 days overdue. Who can take care of this? |
Calm down melvin, waiting |
Out for review |
@blimpich Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Finished and deployed. |
See this comment for context: #46994 (comment)
We have to wait till both of these PRs are merged:
And then we need to quickly do what is outlined here: https://github.com/Expensify/Auth/pull/11946/files#diff-5187b8b526029197fbf1394c069d4a11f7a14dc2cbd436115206dd744aa70befR69
The text was updated successfully, but these errors were encountered: