About the indicator: in the issues comments it has been decided that this button will open a modal instead of a page. For modals we don't have indicators
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Feb 22 2023
Feb 21 2023
Feb 16 2023
Feb 15 2023
Two things:
- I think we need some kind of indicator (like a purple bottom border for Chat/Calendars buttons), seems really confusing to me from the user's perspective that there is no difference between the selected and not selected apps tab. Hover also will be great.
- Look like it will happen soon, but what if there will be a lot of apps? Shouldn't + be sticked to the right side of the container, always visible, and all items scrollable (talking about situation when entire row is filled with apps)? Or at least all items with '+' button scrollable.
Looks great!
(I know there is an example in linear, but I would love a screenshot/video in the summary)
Feb 14 2023
Feb 13 2023
Feb 8 2023
I think the order of diffs in this stack is a little bit mixed, here you use filteredCommunityThreadIDs which is introduced in child revision, but overall looks good
Feb 2 2023
fix typo
Feb 1 2023
update version to 181
Requesting review to get the answer
update message
Jan 31 2023
rename NativeAndroidAccessible -> NativeAndroidAccessProvider
Jan 30 2023
remove unused imports after rebase
make initialize part of initializeQueryExecutor
move call into attemptDatabaseInitialization and test notifs
rename source
rename source
rebase
add missing #include <mutex> and comment
use throw instead of terminate, add SQLiteQueryExecutor::initialize to DatabaseManager::initializeQueryExecutor
Jan 27 2023
Jan 26 2023
rebase before landing
rebase before landing
rebase before landing
rebase before landing, base policy fetchers on viewer.id instead of entire object
Jan 25 2023
improve code
re-requesting review to confirm my understanding of setNewSession is correct
avoid setting field if it not exists
Jan 23 2023
Jan 20 2023
update naming, add source, handle refetchUserData exceptions, improve code
update promises handling
wrap promise inside Promise.all, add notAcknowledgedPolicies to SIWE responder, update LogInResponse fields to be read only
Jan 12 2023
refactor promises
rebase
EDIT: tested for SIWE
In D6092#183060, @ashoat wrote:We care only about logging in manually performed by the user.
Can you remind me why that is?
rely on LOG_IN_SUCCESS action
Jan 10 2023
rebase
Jan 9 2023
avoid repeated code