Skip to content
This repository has been archived by the owner on Jun 30, 2022. It is now read-only.

Added Slack profile back into metadata #80

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ginja
Copy link
Contributor

@Ginja Ginja commented Feb 17, 2016

Fixes hmset issue in #57. The hmset method expects an even length Array. Any empty array in the metadata hash will be ejected when it's flattened.

Fixes hmset issue in litaio#57. The hmset method expects an even length
Array. Any empty array in the metadata hash will be ejected when it's
flattened and will result in an uneven Array.
@jimmycuadra
Copy link
Contributor

Sorry for the delay in responding — can you add a test case that verifies this behavior?

@Ginja
Copy link
Contributor Author

Ginja commented May 11, 2016

Sorry my delay! I will attempt to add a test when I find some time.

maxvt pushed a commit to maxvt/lita-slack that referenced this pull request Oct 30, 2016
This builds on litaio#80
which has not received a response by the original committer.
maxvt pushed a commit to maxvt/lita-slack that referenced this pull request Oct 30, 2016
This builds on litaio#80
which has not received a response by the original committer.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants