Open ribose-jeffreylau opened 7 years ago
Right now, there exists several inconsistencies with the documentation and the implementation. Here are some of the things to be done:
For both Ruby and JS:
encode_sync_str
decode_sync_str
IdPacker
UuidPacker
base_string
en
sync_str
LZString
,
For JS only,
range
Right now, there exists several inconsistencies with the documentation and the implementation. Here are some of the things to be done:
For both Ruby and JS:
encode_sync_str
&decode_sync_str
fromIdPacker
to a more common namespace, since it also works for UUID as well as integer ID.UuidPacker
's interface more intuitive (base_string
can be an instance variable, etc).en
/decode_sync_str
for UUIDs (i.e.sync_str
works for UUID too).UuidPacker
's algorithm to compress/decompress UUID inen
/decode_sync_str
instead of usingLZString
and concatenating with,
.For JS only,
range
class is missing. Need to include its implementation somewhere. (handled by @PeterTKY / @ribose-jeffreylau)