response.status = rate-limit is not actually used anywhere #94

Closed
opened 2025-11-09 09:40:33 -06:00 by GiteaMirror · 1 comment
Owner

Originally created by @dumbmoron on GitHub (Aug 12, 2023).

bd04de65d7/src/modules/sub/utils.js (L23-L24)

bd04de65d7/docs/API.md (L33)

should probably drop it or start using it in next api update, since it might be somewhat misleading if it's mentioned in docs, but the response when ratelimited has .status == 'error':

bd04de65d7/src/core/api.js (L38-L41)

Originally created by @dumbmoron on GitHub (Aug 12, 2023). https://github.com/wukko/cobalt/blob/bd04de65d718fd0c6fe8a455fee28fb40efcde85/src/modules/sub/utils.js#L23-L24 https://github.com/wukko/cobalt/blob/bd04de65d718fd0c6fe8a455fee28fb40efcde85/docs/API.md?plain=1#L33 should probably drop it or start using it in next api update, since it might be somewhat misleading if it's mentioned in docs, but the response when ratelimited has .status == 'error': https://github.com/wukko/cobalt/blob/bd04de65d718fd0c6fe8a455fee28fb40efcde85/src/core/api.js#L38-L41
Author
Owner

@wukko commented on GitHub (Aug 15, 2023):

fixed in 7.0 :)

@wukko commented on GitHub (Aug 15, 2023): fixed in 7.0 :)
Sign in to join this conversation.