assaf / vanity

Experiment Driven Development for Ruby
http://vanity.labnotes.org
MIT License
1.55k stars 269 forks source link

Getting an error from vanity when trying to update redis #347

Closed toddmetheny closed 5 years ago

toddmetheny commented 6 years ago

We just updated our redis instance.

Now we're getting an error any place we have a call to vanity.

For example:


      mixpanel.track('Acquisition Event', {
        'Trigger Type': '8 seconds',
        'Show New Version': <%= ab_test(:sg_20170624_homepage_redesign) %>,
        'Page': 'Homepage'
      })
    }, 8000); // 8 seconds```

The error points to the ab_test line and reads as: 

```>> ab_test(:sg_20170624_homepage_redesign)
!! #<Redis::CommandError: ERR unknown command '[]'>```
phillbaker commented 6 years ago

Hey! f06d90b5bddb2c91df44a79d6c3070c57080e7da on master should fix the issue. Haven't done a release yet, but if you could use master for a bit, that's probably the quickest fix.

phillbaker commented 5 years ago

@toddmetheny did master fix the issue?

toddmetheny commented 5 years ago

@phillbaker Ah---I missed that comment. We switched to use split instead. Wish I had seen it.