stephpy/timeline-bundle

Spread time on Provider

Closed this issue · 2 comments

As @adrienbrault said on:

e530e5c#commitcomment-1039151

Spread time should not be time(), or can be changed by user.

Actually we have:

highco_timeline:
    provider: highco.timeline.provider.redis    # write your own

I think this is a good evol:

highco_timeline:
    provider: 
        id: highco.timeline.provider.redis    # write your own
        options:
          ..... define options there (spread time or others)

Actually only one options to define on redis provider (pipeline), this options can be set on parameters:

Spread time can be override on TimelineAction

I don't think we should make a BC break by using this today:

highco_timeline:
    provider: 
        id: highco.timeline.provider.redis    # write your own
        options:
          ..... define options there (spread time or others)

We should keep this for larger changes