Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
elevchyt created a review comment on a pull request on adsviewer/turboviewer
There's no need for this at the moment
elevchyt created a review comment on a pull request on adsviewer/turboviewer
it's used inside [comment-input.tsx](https://github.com/adsviewer/turboviewer/pull/702/files/340f8073569f5edb793b532ca85cc2ddfbf9485e)
elevchyt created a review comment on a pull request on adsviewer/turboviewer
yeah agree
elevchyt created a review on a pull request on adsviewer/turboviewer
These are fine by me
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- more type fixes 340f807
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- more lint fixes 7965349
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- fix packages 1229407
elevchyt pushed 24 commits to 685-notifications-alerts-system adsviewer/turboviewer
- 703: Fetch ad previews when group by creativeId (#704) fc3d33b
- 706: spend is divided 2 times by 100 (#707) c7b948d
- Auto-update packages (#708) * Auto-update packages * regenerate pnpm-lock.yaml --------- Co-authored-by: Ad... 13e1131
- db stuff a1b8be6
- optional comment mention creative id f71da94
- receivingUserId rename 12f93e6
- endpoints 464acce
- notifications 406f373
- subscription implementation 2a05f80
- extra data in notification 2a06cb2
- prims json types db99216
- main notifications flow without subscription implementation on FE 3ffa87f
- translations ed3fd15
- style select fix 6505d21
- mark notifications as read on hover individually f735ba8
- allow iframe fetching with creative id 4575b6c
- main notification to mention comments flow and view 6c4aa0d
- new integration notification handle 2df7471
- comment for later 1857382
- extra data handling for comment mention redirect 5e67fc8
- and 4 more ...
elevchyt created a review comment on a pull request on adsviewer/turboviewer
```suggestion if (!root.adId || !root.creativeId) return null; ``` I haven't tested it, but shouldn't it be || instead of &&?
elevchyt created a review comment on a pull request on adsviewer/turboviewer
```suggestion !isPending || ```
elevchyt created a review comment on a pull request on adsviewer/turboviewer
I haven't tested it, but shouldn't it be || instead of &&?
elevchyt opened an issue on adsviewer/turboviewer
subscription for changes in organization data
We need a subscription that listens for changes in the organization data (e.g. new users) and make the client update their organization data atom values. This will simplify things a lot because we...elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- mention org users 13c51a5
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- tiptap editor for comments, implemented with mantine custom styles 3e4a3c9
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- insight cards same height improvement & added control of rendering individual components of instight card 73772d2
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- extra data handling for comment mention redirect fe7036f
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- comment for later 4ab186c
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- new integration notification handle d545875
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- main notification to mention comments flow and view 10f355b
elevchyt pushed 1 commit to 685-notifications-alerts-system adsviewer/turboviewer
- allow iframe fetching with creative id 6cde9ba
elevchyt pushed 18 commits to 685-notifications-alerts-system adsviewer/turboviewer
- 655: introduce threshold in insights (#694) 72c9e49
- Fix: added right external id for getUserId method (#696) f112e91
- add useSentry plugin in yoga 207ceb3
- Fix: fixed get ad previews for google (#698) 97b7513
- 699: filter insights by creative ids (#700) a925bb1
- Fix: added the right embed url for iframe (#701) c47a0df
- db stuff 8d96718
- optional comment mention creative id 08783f7
- receivingUserId rename 38bfd46
- endpoints b88de34
- notifications d17b3e5
- subscription implementation 0314eed
- extra data in notification ed5e141
- prims json types 16fba02
- main notifications flow without subscription implementation on FE ef86b81
- translations 8351675
- style select fix 7f71bff
- mark notifications as read on hover individually 1f90097