-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathTODO.txt
34 lines (31 loc) · 1.36 KB
/
TODO.txt
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
MARKETING
[OK] Update favicon
[] Update README.md
[] Review landing page?
FUNCTIONALITY
[OK] Run tests on github actions
[OK] Move /dashboard/lists to /dashboard page
[OK] By default select the first list
[] Add error state for all components
[OK] When creating a new list (ListModal)
[] When fetching fails (also check background fetch of react-query)
[] Add missing handlers for msw (utils/mocks/handlers.js)
[] Review if any endpoint is not returning all results (need to review pagination)
[] Add expiration to cookies
[] When getting an error for any Twitter request related to wrong credentials we should remove cookies
[] Migrate all endpoints to twitter-lite
[] Use id_str in api/user (check this for all endpoints, DRY code)
[] Handle not owned lists (you can't edit, unsubscribe instead of delete, can't add members)
[] Add validation when deleting list with more than x members
[] Handle not authenticated in api
NICE TO HAVE
[] Review if ESLint config is working (react/jsx-key is not showing error)
[] Improve error logging (front + back-end)
[] Add google analytics
[] Improve AddMembersDrawer style (add loading state)
[OK] Fix repeated map for lists of members (add types)
[OK] Review Twitter permissions asked
[] Add modal when trying to delete list with more than n members
CONFIG
[] Fix API requests in vercel preview links
[] Add Prettier and rules to order imports