App icon indicating copy to clipboard operation
App copied to clipboard

[Domain Control] Unify the logic for checking if to queue onyx update for domain admin in domain

Open mountiny opened this issue 4 weeks ago • 3 comments

Part of https://github.com/Expensify/App/issues/76750

Currently, we are only sending the domain_ onyx keys for normal domains and not work workspace feed accounts (that are also technically domain accounts).

This means we need a centralised check that we can use in all these flows that we do not forget to skip the workspace feed in these updates

At the same time, we should make the App code safe not to crash when the domain key lacks some data and handle that gracefully.

mountiny avatar Dec 19 '25 12:12 mountiny

Whoops! This issue is 2 days overdue. Let's get this updated quick!

melvin-bot[bot] avatar Dec 23 '25 00:12 melvin-bot[bot]

Eep! 4 days overdue now. Issues have feelings too...

melvin-bot[bot] avatar Dec 24 '25 23:12 melvin-bot[bot]

Created a PR:

  • https://github.com/Expensify/Auth/pull/18948

rayane-d avatar Dec 25 '25 19:12 rayane-d

@rayane-d Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Jan 02 '26 00:01 melvin-bot[bot]

PR is ready for review

rayane-d avatar Jan 05 '26 11:01 rayane-d