Claire Peng

@raclim @khanniie Just saw this in the open issues when I searched 'typescript', and I thought this was a great idea! Should we add this to the goals for server...

@ksen0 tagging in case this got buried after the rate-limiting stuff last week! The associated PR to resolve this is what lead me to spot the rate-limiting issue originally haha...

Hi @ksen0, tagging you for visibility! Let me know what you think

Hi both happy to work on the fix for the refactor! Will probably have some time this weekend I think we might need to update `api/curation/sketches` to return some extra...

@ksen0 - I'll take a look to see if we can remove the `/code` hits & any other API-call optimisations we can do will solve the issue, then see if...

Hi @ksen0 @msawired @limzykenneth I have opened a PR to resolve the pipeline block here: https://github.com/processing/p5.js-website/pull/838 I looked into removing the call to `/code`, and saw that it was being...

@msawired That makes sense for the width/height thanks! Maybe we just use the [default iframe](https://github.com/processing/p5.js-website/blob/main/src/layouts/SketchLayout.astro#L81-L87) in that case? Would it be possible to also add `fullname` to `/api/sketch/:id`? These would...

Ah ok, in that case maybe the changes on the PR as-is is sufficient? Builds will still make the call to `/code` to extract the `height` and `length`, but we've...

Thanks @ksen0! Happy to do the path for `2.0` too, I should have some time in the eve today (BST). @msawired `license` and `fullname` on the respective endpoints above would...

Hi @ksen0 and @msawired, I've updated the current issue with the small clean-up tasks, and created the new issue for updating to use the bearer-token [here](https://github.com/processing/p5.js-website/issues/845)