Elias W. BA
Elias W. BA
The only place I can see an error happening that the worker wouldn't be aware of is when we maybe refresh the credential before sending the job to the worker....
@elias-ba when reviewing, please focus on (1) sensitive data we may send to the worker (2) does this pr cover changes brought in later (like refresh flow).
Closing in favor of #2289 2289
Closing now as the intent of this code was implemented in #2289 in July. It was reopened accidentally.
### Review / change requests progress - [x] Remove default validation on the text fields when a modal loads - the data validation should only show when a user tries...
>  > > @christad92 @elias-ba, some feedback so far 👆 > > Also if I add only one scope (as in type without entering a `,`) and click save...
@taylordowns2000 this looks like a valid way to implement this. I am not sure I can't validate it's relevancy but it seems to me as a better way of dynamically...
The fix for this issue is partly implemented here: #2149. But it's blocked by #1825.
@christad92 I would say partly fixed it. But we will probably come back to it for pure engineering review / optimisation.
The solution to this problem seems much more expensive than the current budget. I spoke with @stuartc and @christad92, and we all think it's better if I solve the most...