Corey Martin

Results 10 comments of Corey Martin

Test comment

Test comment

Thanks for this! I gave it a test and the X is rendering a bit below the title bar on the latest Bootstrap. Feel free to take a look by...

Thanks! Make sure you go to /pages/home and create an account and login, since announcements are shown only to logged in users On Sun, Sep 20, 2015 at 7:47 PM...

This is fantastic and I'll work to incorporate it into master. Performance is certainly critical and saving read announcements with a cookie if someone isn't logged in opens up new...

Fantastic. Your optimizations will help get this to 1.0

I incorporated indexing and uniqueness validation for 1.0. I plan to bring your feature for non-user-facing announcements into a later version - your implementation is really well-executed.

Keeping this open, as I'd like to get your performance enhancements into the next minor version.

Redis support is a great idea. I would welcome a pull request to make this happen. I definitely see your point about implementation cost. In practice, Starburst is in use...

A very belated yes!