Skip to content

Savateykin #314

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
yaroslavsavateykin wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Savateykin #314

Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
2e80aff
atm
Nov 5, 2025
5ab79ab
sdjkfldsj
Nov 5, 2025
cca87d3
adsjdsfjlk
Nov 5, 2025
f08088c
Merge branch 'tmp'
Nov 5, 2025
216818b
additional_data json type check
Nov 10, 2025
582078a
additional_data json check
Nov 10, 2025
582795e
additional_data json check
Nov 10, 2025
bd716cb
additional_data json check
Nov 10, 2025
32219fe
jsonb_valid function usage
Nov 28, 2025
7e3d45e
jsonb_valid usage
Nov 28, 2025
9fbc9e6
regular expression json check
Dec 5, 2025
b150d84
jsonb_valid removal
Dec 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions dags/ods/marketing/printer_bots_actions.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,12 @@ FROM
LATERAL (
SELECT value AS elem
FROM jsonb_array_elements(
CASE
WHEN additional_data = '' THEN jsonb_build_array('{}'::jsonb)
CASE
WHEN additional_data IS NULL OR additional_data = '' THEN jsonb_build_array('{}'::jsonb)
--WHEN NOT jsonb_valid(additional_data) THEN jsonb_build_array('{}'::jsonb)
Copy link
Contributor

@VladislavVoskoboinik VladislavVoskoboinik Dec 12, 2025

Choose a reason for hiding this comment

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

убери комментарий

WHEN additional_data ~ '^\{.*\}$' THEN additional_data::jsonb
WHEN jsonb_typeof(additional_data::jsonb) = 'array' THEN additional_data::jsonb
ELSE jsonb_build_array(additional_data::jsonb)
ELSE jsonb_build_array(additional_data::jsonb)
Copy link
Contributor

@VladislavVoskoboinik VladislavVoskoboinik Dec 12, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

а чем 31 и 32 строчки отличаются по смыслу?
сейчас написано: если массив, то преобразуем в json

иначе преобразуем в массив джейсонов

точно ли это имел в виду?

END
)
) AS expanded
Expand Down