Skip to content

Metadata a#84

Open
Y-PLONI wants to merge 1 commit into
kdroidFilter:masterfrom
Otzaria:metadata_A
Open

Metadata a#84
Y-PLONI wants to merge 1 commit into
kdroidFilter:masterfrom
Otzaria:metadata_A

Conversation

@Y-PLONI

@Y-PLONI Y-PLONI commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

ענף זה נועד לטפל בדבר אחד בלבד:
שגיאה בתהליך ייבוא המטא-דאטה, כפי שכתבתי לך בslack:
העמודה ב‑DB נקראת heShortDesc (תיאור קצר), אבל בפועל היא מקבלתאת heDesc (התיאור הארוך).

הPR נכתב על ידי המתכנתת, ורשום על שמי בגלל הrebase.


שים לב:
השתרבב קומיט אחד שמבצע מיגרציה לdelta - במקרה שביעד חסרות טבלאות.
אם זה לא מוצא חן בעיניך - אפילו קצת - תסגור מיד ואגיש אותו בPR נפרד!!!

@kdroidFilter

kdroidFilter commented Jun 26, 2026

Copy link
Copy Markdown
Owner

תודה רבה, אבל לא צריך לעשות migration
זית לא מפעיל עדכוני דלטא, ואתם כנראה גם לא משתמשים, אחד הסיבות שאני לא רוצה עדכוני דלטא כי אני לא רוצה migration בקוד
בשבילי הרבה יותר פשוט לעשות את זה #87
אבל אני ככל הנראה לא אשתמש יותר בCI של גיטהוב אז כרגע גם זה לא רבלנטי בשבילי

@Y-PLONI

Y-PLONI commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

@kdroidFilter
שום בעיה, רק להיות ברור: להסיר את הקומיט הזה?
אשמח אם תעבור בנתיים על הקומיט הראשי, הוא מאוד קטן - והוא הכנה להוספת הmetadata של אוצריא, ושינוי המידע הבעייתי מספריא.

@YOSEFTT

YOSEFTT commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

שום בעיה, רק להיות ברור: להסיר את הקומיט הזה?

מה לא הבנת?
כן, תסיר, הוא כתב לך את זה במפורש!!

Collapse the 5-line KDoc into a single line, matching the file's sparse comment style.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Y-PLONI

Y-PLONI commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

@kdroidFilter
הcommit הוסר: נשאר לך לבדוק כמה שורות...

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