Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
Fraggle closed a pull request on dust-tt/dust
Update nginx to use our own image
# Description This PR updates the nginx configuration to use our own custom image. Changes include: - Adding a new `.gcloudignore-nginx` file - Creating a new GitHub Actions workflow for de...JulesBelveze opened a pull request on dust-tt/dust
[sparkle] - enh: `Button`
## Description <!-- Briefly describe the changes you've made and link any relevant issues (e.g., "Fixes #123"). --> <!-- If the PR includes UI changes, please attach a screenshot or GIF to illu...JulesBelveze pushed 4 commits to sparkle/enh/button dust-tt/dust
- [sparkle] - refactor: update Button.tsx and related story - Add 'as const' assertion to BUTTON_VARIANTS for more pr... 4e06f84
- [sparkle] - refactor: rename variant variables for button component - Rename `variantStyle` to `styleVariants` for ... 2bed134
- [sparkle] - refactor: streamline IconButton with Button component - Replace custom IconButton implementation with B... 3866401
- [sparkle] - refactor: standardize button variant use across components - Update several IconButton uses from 'secon... 65c8417
flvndvd created a review comment on a pull request on dust-tt/dust
Not sure I'm a big fan of one big file, it's pretty hard to parse, and maintain. Can we maybe have one file per resource and use a "barrel file" to export them all?
flvndvd created a review comment on a pull request on dust-tt/dust
`nit` But if we plan to expose this method publicly I'd go with `request` to align on the most common libraries.
flvndvd created a review comment on a pull request on dust-tt/dust
`nit` We can destructure `data`.
flvndvd created a review comment on a pull request on dust-tt/dust
Wait we were not checking this on the Public API 🤔?
flvndvd created a review comment on a pull request on dust-tt/dust
It's actually the query 😉
flvndvd created a review comment on a pull request on dust-tt/dust
Why not using zod to handle this parsing?
flvndvd created a review comment on a pull request on dust-tt/dust
Don't you need to pass a status as well?
flvndvd created a review comment on a pull request on dust-tt/dust
We are loosing the original error message here
flvndvd created a review comment on a pull request on dust-tt/dust
We are loosing the original error message here...
flvndvd created a review comment on a pull request on dust-tt/dust
This seems very strange, throwing a custom `Err` type 🤔
flvndvd created a review comment on a pull request on dust-tt/dust
`nit` Let's always keep `auth` as the first argument when present.
flvndvd created a review comment on a pull request on dust-tt/dust
Why do you need the cast here?
flvndvd created a review comment on a pull request on dust-tt/dust
We have a `removeNulls` helper.
flvndvd created a review comment on a pull request on dust-tt/dust
IIRC, you can directly call the `getAgentConfigurations` with all the ids to avoid the need for a promise.all wrapper here.
flvndvd created a review on a pull request on dust-tt/dust
Left a first batch of comments.
flvndvd created a review on a pull request on dust-tt/dust
Left a first batch of comments.
tdraier pushed 1 commit to main dust-tt/dust
- [connectors] Handle 'Bad Request' from google oauth (#8144) 7f05406