Skip to content
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.

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

Add content type for copy cataloging #73

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

Conversation

acka47
Copy link
Contributor

@acka47 acka47 commented Dec 17, 2024

@acka47 acka47 force-pushed the addFremddatenContentType branch from 000182f to 3dee532 Compare December 17, 2024 13:28
@acka47 acka47 marked this pull request as ready for review December 17, 2024 13:28
@acka47 acka47 requested a review from fsteeg December 17, 2024 13:28
@acka47
Copy link
Contributor Author

acka47 commented Dec 17, 2024

Ich habe jetzt weder die Darstellung und UI-Feldnamen konfiguriert noch, dass das RPB-ID-Feld automatisch befüllt wird und nicht editiert werden kann. Sollte das auch im Kontext dieses PRs passieren?

@fsteeg
Copy link
Member

fsteeg commented Dec 17, 2024

Ich habe jetzt weder die Darstellung und UI-Feldnamen konfiguriert noch, dass das RPB-ID-Feld automatisch befüllt wird und nicht editiert werden kann. Sollte das auch im Kontext dieses PRs passieren?

Ne, passt, da hole ich vorher nochmal die aktuelle Konfig vom Produktivsystem.

@fsteeg
Copy link
Member

fsteeg commented Dec 18, 2024

Sieht doch im Groben schon mal gut aus. Zwei Punkte die mir auffallen:

  • Du hast die Umbenennung der .env.example committed (kopieren zu .env wäre das Richtige gewesen – du könntest insgesamt auf Basis deiner Erfahrung noch das README updaten wegen env, sudo bzw. Phu's Link im Chat um sudo zu vermeiden und nötiges npm install bzw. ein Issue damit das nicht mehr nötig ist?)
  • hbzId sollte dann ein Lookup-Feld sein, oder? Das kannst du ja auch im Content-Type-Builder einstellen und die Quelle konfigurieren (Neues Feld -> oben rechts Tab 'Custom')

Dann könnte ich das Befüllen der rpbId umsetzen und direkt das Holen und testweise Indexieren der angereicherten Lobid-Daten hier im PR probieren, das sollte dann eigentlich nicht mehr viel sein (wir machen ja fast das gleiche schon).

@fsteeg fsteeg assigned acka47 and unassigned fsteeg Dec 18, 2024
@acka47 acka47 force-pushed the addFremddatenContentType branch from 3dee532 to 328ccef Compare December 18, 2024 11:46
@acka47
Copy link
Contributor Author

acka47 commented Dec 18, 2024

Du hast die Umbenennung der .env.example committed (kopieren zu .env wäre das Richtige gewesen

Habe das --force-with-lease gepusht.

du könntest insgesamt auf Basis deiner Erfahrung noch das README updaten wegen env, sudo bzw. Phu's Link im Chat um sudo zu vermeiden und nötiges npm install bzw. ein Issue damit das nicht mehr nötig ist?

Habe 24276d3 ergänzt und #74 angelegt.

hbzId sollte dann ein Lookup-Feld sein, oder? Das kannst du ja auch im Content-Type-Builder einstellen und die Quelle konfigurieren (Neues Feld -> oben rechts Tab 'Custom')

Das mache ich morgen oder im neuen Jahr. Gerade kann ich – wie bereits im Chat beschrieben – den Content Type Builder lokal nicht aufrufen: Er erscheint eine halbe Sekunde lang, dann wird die Seite weiß.

Jetzt geht es aber erstmal zur Weihnachtsfeier.

@acka47 acka47 assigned fsteeg and unassigned acka47 Jan 6, 2025
@fsteeg
Copy link
Member

fsteeg commented Jan 6, 2025

Gerade kann ich den Content Type Builder lokal nicht aufrufen: Er erscheint eine halbe Sekunde lang, dann wird die Seite weiß.

Hier klappt es in deinem Branch. Kommt denn was im Log? Ansonsten mal mit --build starten:

docker compose up --build

@fsteeg
Copy link
Member

fsteeg commented Jan 7, 2025

Ich habe die Lifecycle-Hooks für ID-Vergabe, Backup/Export, Labels und Indexierung sowie die Links in der Info-Box eingerichtet (hier im PR, siehe Commits oben) und die Zusammenführung mit dem Lobid-Datensatz vor der Indexierung umgesetzt (in der PUT-Route der rpb-App, s. hbz/rpb#118).

Auf Test deployed:

View-Config kannst du nach Belieben auf Test anpassen (werde ich sowieso manuell auf Prod übertragen).

@fsteeg fsteeg assigned acka47 and unassigned fsteeg Jan 7, 2025
@acka47 acka47 assigned fsteeg and unassigned acka47 Jan 7, 2025
@fsteeg fsteeg force-pushed the addFremddatenContentType branch from 51867da to 2f79228 Compare January 28, 2025 17:29
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.

2 participants