FF-2176: Add new earning preset #136
Conversation
presets/russia.yaml
Outdated
| right: true | ||
| - left: typespecs | ||
| operation: has_none_of | ||
| right: foreign |
There was a problem hiding this comment.
{'filter': [{'left': 'typespecs', 'operation': 'has_none_of', 'right': 'foreign-issuer'}]}
There was a problem hiding this comment.
Выяснил. Нет в этом необходимости. is_primary достаточно.
presets/uk.yaml
Outdated
| - left: is_primary | ||
| operation: equal | ||
| right: true | ||
| - left: typespecs |
There was a problem hiding this comment.
{
'filter': [{
'left': 'typespecs',
'operation': 'has_none_of',
'right': 'cdi'
}, {
'left': 'exchange',
'operation': 'equal',
'right': 'LSE'
}, {
'left': 'is_primary',
'operation': 'equal',
'right': True
}]
}
There was a problem hiding this comment.
Добавил 'has_none_of' => 'cdi'.
presets/thailand.yaml
Outdated
| - left: is_primary | ||
| operation: equal | ||
| right: true | ||
| - left: typespecs |
| @@ -606,3 +606,18 @@ presets: | |||
| sort: | |||
| sortBy: description | |||
| sortOrder: asc | |||
| - name: earning | |||
| filter: | |||
There was a problem hiding this comment.
По какой-то причине в проде кстом фильтры не применяются для Канады Индии Японии, только для России Тайланда, ЮК, Бразилии
if self.category['region'] in ["ru", "br", "uk", "th_TH"]:
EXTRA_FILTERS: t.Dict[str, t.List[t.Dict[str, t.Any]]] = {
"ca": [
{'left': 'exchange', 'operation': 'in_range', 'right': ['TSX', 'TSXV', 'CSE']},
],
"in": [
{'left': 'exchange', 'operation': 'in_range', 'right': ['NSE']},
],
"ja": [
{'left': 'exchange', 'operation': 'in_range', 'right': ['TSE']},
],
"ru": [
{'left': 'typespecs', 'operation': 'has_none_of', 'right': 'foreign-issuer'}
],
"uk": [
{'left': 'typespecs', 'operation': 'has_none_of', 'right': 'cdi'},
{'left': 'exchange', 'operation': 'equal', 'right': 'LSE'},
],
"th_TH": [
{'left': 'typespecs', 'operation': 'has_none_of', 'right': 'foreign'}
],
"br": [
{'left': 'typespecs', 'operation': 'has_none_of', 'right': 'odd'}
],
}
Это тоже надо уточнить со скриенрцами баг или небаг
Если небаг, то надо убрать уточнения из фильтров Канады Индии Японии
There was a problem hiding this comment.
По Индии, оставляем потому-что
"скорее всего, стоит добавить эти фильтры, в индии две биржи , символа на них идентичны, одни символа для резидентов и для нерезидентов, вроде
так что по ним будет приходить одно и то же и одно мы выбрасываем, считая это дублями"
По Канаде и Японии
"можно сделать так же аналогично другим маркетным страницам по выбранным биржам"
Так что оставляем, чтобы было как и на других маркетных страницах, раз уж у нас только на это не работает.
|
Посмотрел, у меня вопросов больше нет |
|
По global.yaml тоже ок |
|
Коммит FF-2176: Remove NSE exchange from India earning preset — любо! можно мержить |
No description provided.