table_sync icon indicating copy to clipboard operation
table_sync copied to clipboard

Ability to specify name of model when publishing raw data

Open AnotherRegularDude opened this issue 4 years ago • 1 comments

Changes

  • Add model_name attribute to the raw publishing interface. So now you can specify model name other then object class name when using raw method.
  • Remove object_class attribute from TableSync::Publishing::Raw. Need to use model_name instead.
  • Add notify settings to the global config: if notify is false, or nil, we don't send notifications, even if notifier setting is set.
  • TableSync::Publishing::Raw accepts two additional attributes: table_name and schema_name. These attributes are passed to the notify function if notify = true.
  • TableSync::Instrument#notify now validates incoming args.

AnotherRegularDude avatar Jan 18 '22 16:01 AnotherRegularDude

О, классная фича, можно будет у нас костылёк убрать 👍

tycooon avatar Jan 18 '22 16:01 tycooon

@VegetableProphet, го дропнем руби 2.5 хотя бы (я б, канеш, и 2.6 дропнул), у них уже еол

Я прост фиг знает, как убрать required с джобки по 2.5: у меня нет доступа к настройкам

AnotherRegularDude avatar Nov 24 '22 21:11 AnotherRegularDude

@AnotherRegularDude а они просто в CI-матрице в CI.yml лежат давайте 2.5 дропнем, уже абсолютно устарел

0exp avatar Nov 24 '22 22:11 0exp

2.6 я бы в январе дропнул (он уже EOL, но секурити фиксы в этом году еще выходят вроде как. у нас вроде наименьший релиз - 2.7 на проектах, но таких единицы)

0exp avatar Nov 24 '22 22:11 0exp

@AnotherRegularDude а они просто в CI-матрице в CI.yml лежат давайте 2.5 дропнем, уже абсолютно устарел

А я дропнул

Но вон 2.5 Required пишет

Видимо, это в настройках правится

AnotherRegularDude avatar Nov 28 '22 08:11 AnotherRegularDude