-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Implement hasStrictAdvancedApproval
#47517
Comments
strictworkFlow
check when editing submitted reports in ControlhasStrictAdvancedApproval
@youssef-lr Whoops! This issue is 2 days overdue. Let's get this updated quick! |
On it this week |
@youssef-lr Eep! 4 days overdue now. Issues have feelings too... |
Will try to to get to this this week |
@youssef-lr Eep! 4 days overdue now. Issues have feelings too... |
@youssef-lr Still overdue 6 days?! Let's take care of this! |
No update yet, will get to this soon this week or Monday |
Will get to this when I'm done with the Auth PR for adding expenses to submitted reports in Control |
@youssef-lr Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@youssef-lr 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
On it this week |
@youssef-lr Eep! 4 days overdue now. Issues have feelings too... |
@youssef-lr 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
No update yet |
@youssef-lr Eep! 4 days overdue now. Issues have feelings too... |
No update yet |
@youssef-lr Eep! 4 days overdue now. Issues have feelings too... |
@youssef-lr 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
I'm on this next week |
@youssef-lr Eep! 4 days overdue now. Issues have feelings too... |
I have some spare cycles now and can tacke this this week. |
@youssef-lr Huh... This is 4 days overdue. Who can take care of this? |
I got caught up with a customer issue and couldn't get to this yet. |
@youssef-lr Huh... This is 4 days overdue. Who can take care of this? |
No update, I couldn't get to this |
@youssef-lr Eep! 4 days overdue now. Issues have feelings too... |
We don't need this anymore as we're changing the logic of how we check for forwarded reports and it won't matter whether the policy has strict workflow or not. |
Implement
hasStrictAdvancedApproval
similar to what we have in Auth and use it when checking if submitted expenses can be edited in Control.#47398 (comment)
The text was updated successfully, but these errors were encountered: