• Joined on 2023-07-08
zephyr pushed to typespec at ligmo/ligmo 2025-05-06 12:11:23 -04:00
d412014bb7 api: format
0adc9ce683 api: only query emotes by id directly
24d2f09767 api: include connections in emote sets
Compare 3 commits »
zephyr deleted branch cleanup-tooling from ligmo/ligmo 2025-05-06 12:01:56 -04:00
zephyr pushed to typespec at ligmo/ligmo 2025-05-06 12:01:56 -04:00
43cd2af30f move prettier config to root, cleanup api/, and version typespec in package.json (#21)
zephyr merged pull request ligmo/ligmo#21 2025-05-06 12:01:54 -04:00
move prettier config to root, cleanup api/, and version typespec in package.json
zephyr commented on pull request ligmo/ligmo#21 2025-05-06 11:49:03 -04:00
move prettier config to root, cleanup api/, and version typespec in package.json

Didn't you want to remove /api/.gitignore too?

zephyr commented on pull request ligmo/ligmo#21 2025-05-06 11:49:03 -04:00
move prettier config to root, cleanup api/, and version typespec in package.json

This file can be deleted now that we aren't using Elixir.

zephyr commented on pull request ligmo/ligmo#15 2025-05-06 11:09:31 -04:00
schema: adjustments

Now we have different tables named emote_set and set_emote...

zephyr closed issue ligmo/ligmo#16 2025-05-06 10:57:15 -04:00
decide the name of a collection of emotes
zephyr commented on issue ligmo/ligmo#16 2025-05-06 10:57:15 -04:00
decide the name of a collection of emotes

Every open PR now uses "emote set," so the decision is made and implemented.

zephyr commented on issue ligmo/ligmo#16 2025-05-06 10:55:48 -04:00
decide the name of a collection of emotes

I think the fact that both 7TV and BTTV use "emote set" means it's the correct choice by convention. That said, I had chosen list first because it's a much more common word in English. I also…

zephyr pushed to typespec at ligmo/ligmo 2025-05-06 10:53:20 -04:00
e3b61cb574 api: rename emote list -> emote set
4805480051 api: move route tags to op definitions
Compare 2 commits »
zephyr pushed to schema-updates at ligmo/ligmo 2025-05-06 10:38:34 -04:00
ddb5bf4964 schema: rename emote list -> emote set
zephyr pushed to typespec at ligmo/ligmo 2025-05-06 10:23:41 -04:00
371467b616 api: first round of pr feedback updates
zephyr commented on pull request ligmo/ligmo#20 2025-05-06 10:18:35 -04:00
read api specification

I also wonder about pagination. I've never needed to do it with APIs I've written, and I don't know at what sort of scale you generally start to care. TypeSpec has stuff built-in for paginating,…

zephyr commented on pull request ligmo/ligmo#20 2025-05-06 10:10:06 -04:00
read api specification

The DB has ban reason and unban time. As I said in the previous comment, I'll be looking into how to incorporate the latter with the correct visibility rules.

zephyr commented on pull request ligmo/ligmo#20 2025-05-06 08:31:39 -04:00
read api specification

This would also suggest that /users/{platform}/{platformID} should instead be /platforms/{platform}/users/{platformID}. Am I reading that right? Do you agree?

zephyr commented on pull request ligmo/ligmo#20 2025-05-06 08:29:30 -04:00
read api specification

ban is the reason they're banned, not the unban time. We'd only want to show the latter to the user themselves and to moderators. TypeSpec has some stuff to express that, but I have to read more…

zephyr commented on pull request ligmo/ligmo#20 2025-05-06 08:26:30 -04:00
read api specification

Intentional. There's no useful comment to add here.

zephyr commented on pull request ligmo/ligmo#20 2025-05-06 08:22:34 -04:00
read api specification

yeah i have no idea how to do this

zephyr commented on pull request ligmo/ligmo#20 2025-05-06 08:18:08 -04:00
read api specification

Going to wait on #18 to do this, just to avoid any chance of a merge conflict.