Lercerss / graphelier

TFBAG Capstone project 2019-2020
The Unlicense
5 stars 1 forks source link

[#201] Implemented route for news cluster feature #220

Closed alvyn279 closed 4 years ago

alvyn279 commented 4 years ago
alvyn279 commented 4 years ago

Looks good, whatever I had noticed Chirac brought up already. Just wondering if there were constants used in BE for making it more verbose instead of -1 and 1

I think 1 going forward in time and -1 going backwards in time speaks for itself. But good point, im actually gonna include a docstring

Lercerss commented 4 years ago

Could you unify the types for timestamp in the JSON response? You have both string and number