• Joined on 2025-04-28
hamza closed issue ligmo/ligmo#8 2025-05-06 20:04:27 -04:00
Create Go project scaffold
hamza closed issue ligmo/ligmo#17 2025-05-06 20:04:18 -04:00
Setup Go Project
hamza created pull request ligmo/ligmo#22 2025-05-06 20:03:51 -04:00
fixup go env via mise
hamza created branch fixup-go in ligmo/ligmo 2025-05-06 20:03:08 -04:00
hamza pushed to fixup-go at ligmo/ligmo 2025-05-06 20:03:08 -04:00
d60a9f8d5e fixup go env via mise
hamza pushed to cleanup-tooling at ligmo/ligmo 2025-05-06 11:56:57 -04:00
27d7419294 add description to gen:api
hamza pushed to cleanup-tooling at ligmo/ligmo 2025-05-06 11:51:28 -04:00
781cf8d8e7 remove .vscode
hamza pushed to cleanup-tooling at ligmo/ligmo 2025-05-06 11:51:04 -04:00
8dfd3ca285 remove api/.gitignore
hamza commented on pull request ligmo/ligmo#21 2025-05-06 11:50:42 -04:00
move prettier config to root, cleanup api/, and version typespec in package.json

I just looked over the .gitignore, there isn't anything I am going to move over, since the output dir we are using is schema/ and we do in fact want to keep it.

hamza pushed to cleanup-tooling at ligmo/ligmo 2025-05-06 11:38:29 -04:00
bddf702577 move prettier config to root, cleanup api/, and version typespec in package.json
e3b61cb574 api: rename emote list -> emote set
4805480051 api: move route tags to op definitions
371467b616 api: first round of pr feedback updates
Compare 4 commits »
hamza created pull request ligmo/ligmo#21 2025-05-06 11:35:19 -04:00
move prettier config to root, cleanup api/, and version typespec in package.json
hamza created branch cleanup-tooling in ligmo/ligmo 2025-05-06 11:33:52 -04:00
hamza pushed to cleanup-tooling at ligmo/ligmo 2025-05-06 11:33:52 -04:00
55b906779d move prettier config to root, cleanup api/, and version typespec in package.json
hamza commented on pull request ligmo/ligmo#20 2025-05-06 01:21:54 -04:00
read api specification

While AVIF may be supported by all modern browsers, is this something we can also say for certain for any local GUI app that might be integrating with ligmotes? It might be good to still have some sort of compatibility

hamza commented on pull request ligmo/ligmo#20 2025-05-06 01:21:54 -04:00
read api specification

ID of the user should imply this should be a string right? which is what I think the type should be, not User. Since it doesn't make sense to return all of the user metadata for every single emote query, which also requires fetching all of their emote sets. Deep fetching all of the data like this is going to make our HTTP requests incredibly expensive as we scale, and we should try to target minimizing the joins and requests we need to do to fulfill our API contract in most situations, unless all of the user metadata information is always going to be needed whenever an emote is queried.

hamza commented on pull request ligmo/ligmo#20 2025-05-06 01:21:54 -04:00
read api specification

Similar comment as to above

hamza commented on pull request ligmo/ligmo#20 2025-05-06 01:21:54 -04:00
read api specification

Ban is a time that they are unbanned right?

hamza commented on pull request ligmo/ligmo#20 2025-05-06 01:21:54 -04:00
read api specification

missing field comments

hamza commented on pull request ligmo/ligmo#20 2025-05-06 01:21:54 -04:00
read api specification

missing field comments

hamza commented on pull request ligmo/ligmo#20 2025-05-06 01:21:54 -04:00
read api specification

personally, i hate having multiple .gitignores. sometimes there's weird wacky shit that gets ignored by some sub directory gitignore and its stupid to have to figure out which of the potential gitignores in the directory might be doing some wacky shit. i would prefer promoting things here that need to be to the root /.gitignore