search

nim-lang / redis

Official redis wrapper for Nim.
Other
128 stars 35 forks source link

Revert "Allow for subscribing to multiple channels" #29

Closed ThomasTJdev closed 2 years ago

ThomasTJdev commented 2 years ago

Reverts nim-lang/redis#28

@dom96 - I made a mistake in Githubs web UI! This PR fails. The new proc passes the channel-param inside a seq[string] which creates a seq[seq[string]] since it's already a seq[string].

await r.sendCommand("SUBSCRIBE", @[channels])
# => Error: type mismatch: got <AsyncRedis, string, seq[seq[string]]>

should be

await r.sendCommand("SUBSCRIBE", channels)
dom96 commented 2 years ago

Can you fix the new proc instead?

ThomasTJdev commented 2 years ago

Can you fix the new proc instead?

Of course. It was just for the git history.