Daniel (dB.) Doubrovkine
Daniel (dB.) Doubrovkine
> 1. Change var name to `teams_table` for AR, and `teams_collection` for mongoid. > 2. Change var name to `teams_storage_name` > 3. Other name? I think the cleanest and most...
> so, `.store_in = {teams: 'my_table_or_collection_name'} ` ? What if we just dropped `store_in` and did: ``` teams: { name: 'my_table' } ``` This way it's extensible for the future,...
that LGTM
This needs a major version bump. If you have time, consider doing the MongoDB support for renaming the collection as part of this. Try writing/upgrading a trivial bot. I personally...
> Just noticed that more changes needs to be done for this feature, since the db:migrate doesn't pull in the setting of tablename for ActiveRecord. Will continue to work on...
Related #102, in which the user tried to inherit from `ApplicationRecord` instead of `ActiveRecord::Base`. Could be fixed by this too.
The RTM client's `.channels` is something that comes from a [local store](https://github.com/slack-ruby/slack-ruby-client#realtime-client). Since we are not actually starting a client we either need to stub that store in tests or...
It was suggested by @leaverou, btw.
Please contribute!
https://github.com/pixelslip/Colorkit/issues/1