This repository was archived by the owner on Jul 25, 2023. It is now read-only.
[backend] API update discussion #44
Closed
hackertron
started this conversation in
General
Replies: 2 comments 1 reply
-
Beta Was this translation helpful? Give feedback.
0 replies
-
I support Jeo in mixing the content (urls) in the backend because: A |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
discussion points
content.link seems more semantic than url.link
/mix?tags=628983903892&tags=72982772896&tags=628829988261&tags=668827626728829927
Jay points
I understand the points.
calling /mix is fine , under the hood it can call getUrlByTags
randomizing
tbh I don't expect that we will have millions of urls, thus it made sense to me that random url selection can be done on client side. it's not heavy task .
we can do that on backend side too but then on frontend , frontend have to just iterate over the response (json that is returned )
this brings me another question , what if we iterate over complete response, should we just restart the iteration over response again ?
Currently , it's random access like [1,2,3,4] and select random (arr) . so there is repetition . Many times it can show 2,2 . it's possible.
calling url , content.
-We can do but then we are changing specs completely : https://github.com/State-Channel-4/api-spec. most of my url routes and naming comes from the api-spec.
potential changes
changes that I understand from discussion
using
/mix
for discover that returns a json response with url and it's associated ID.calling
url
tocontent
random on backend side.
Beta Was this translation helpful? Give feedback.
All reactions