This has literally no effect
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
May 10 2023
May 9 2023
could you link the figma?
lgtm thanks for fixing this!
May 8 2023
The code looks good, but I actually pitched this idea to @ashoat initially and we talked about finding another solution. Thinking out lout right now, but wouldn't it be possible to decouple this cycle by moving defaultNotificationPayloadValidator out of account-types and possibly into validation-utils? I think this would also be better since there are no other validators in account-utils and account-utils should exclusively just be for types
Does the sizing look right? Can't really tell from the images from the Test Plan, but as long as things looked right during your testing.
I actually just double checked using the inspector tool, and noticed that these new icons are actually a tad bit smaller, will have ted fix this and send me some new icons
May 6 2023
May 2 2023
May 1 2023
patched in the changes and played around with the views and everything looks good, no more bounds being overflown. Thanks for fixing this
Apr 30 2023
rebase before landing
rebase before landing
rebase before landing
rebase before landing
rebase before landing
rebase before landing
rebase before landing
address comments
address comments
Apr 28 2023
rebase
rebase
split diff into two (this is part 2 and part 1 is handeled by D7684)
one more rebase before landing
one more rebase
rebase before landing
rebase before landing
Apr 27 2023
simplify callback
split diff (this is the first part)
address comments
It would be good to explain the reason for the changes here. Are these changes being made because you're going to reuse the factored-out code in a later diff in the stack, so it makes sense to factor out the shared logic here?
address comments/rebase before landing
rebase before landing
rebase before landing
rebase before landing
rebase before landing