Skip to content

Allow a single index for all events (remove monthly index)#199

Merged
sergehuber merged 1 commit intoapache:masterfrom
YotpoLtd:data_3381
Sep 22, 2020
Merged

Allow a single index for all events (remove monthly index)#199
sergehuber merged 1 commit intoapache:masterfrom
YotpoLtd:data_3381

Conversation

@lyogev
Copy link
Copy Markdown
Contributor

@lyogev lyogev commented Sep 21, 2020

Allow a single index for all events (remove monthly index)

#39)

* feat(DATA_3381-merging-monthly-indices): remove itemsMonthlyIndexed from blueprint, put it in config instead

* feat(DATA_3381-merging-monthly-indices): sending none if monthlyIndex should contain nothing
Copy link
Copy Markdown
Contributor

@sergehuber sergehuber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, looks good to me !

@sergehuber sergehuber merged commit 0192804 into apache:master Sep 22, 2020
amitco1 added a commit to YotpoLtd/unomi-releases that referenced this pull request Oct 21, 2020
#39) (apache#199)

* feat(DATA_3381-merging-monthly-indices): remove itemsMonthlyIndexed from blueprint, put it in config instead

* feat(DATA_3381-merging-monthly-indices): sending none if monthlyIndex should contain nothing

Co-authored-by: amitco1 <amit@yotpo.com>
asfgit pushed a commit that referenced this pull request Nov 11, 2020
#39) (#199)

* feat(DATA_3381-merging-monthly-indices): remove itemsMonthlyIndexed from blueprint, put it in config instead

* feat(DATA_3381-merging-monthly-indices): sending none if monthlyIndex should contain nothing

Co-authored-by: amitco1 <amit@yotpo.com>
(cherry picked from commit 0192804)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants