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

feat: link contact_person (LAN-699) #226

Closed
wants to merge 17 commits into from

Conversation

diePuppe
Copy link
Contributor

Hey Raffael,

kannst du mir kurz weiterhelfen?
Ich kriege die Throw Funktion nicht.

Failed to get method for command landa.organization_management.doctype.landa_member.link_contact_person with module 'landa.organization_management.doctype.landa_member' has no attribute 'link_contact_person'

Weiß, irgendwie nicht mehr weiter woran es liegen könnte

@diePuppe diePuppe changed the title feat: LAN-699 link contact_person feat: link contact_person(LAN-699) Jun 28, 2023
Copy link
Member

@barredterra barredterra left a comment

Choose a reason for hiding this comment

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

Achte bitte auf die korrekte Formatierung. Du kannst dir auch pre-commit im Verzeichnis apps/landa installieren, damit das automatisch bei jedem Commit gemacht wird.

diePuppe and others added 3 commits June 29, 2023 18:53
…r.js

Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
@diePuppe
Copy link
Contributor Author

Hey Raffael,
Funktion ist erstmal nur für die Adresse gegeben, da die (jedenfalls die erste) immer gleich beschriftet wird.
Bei Contact sieht Das schon schwieriger aus, da der Name über die Eingabe des Users vergeben wird.
Hast du noch einen Ansatz, wie man auf das HTML zugreifen kann, obwohl sich der Name aus diesem heraus ergibt?
Außerdem habe ich noch das Problem, dass die Adresse Änderung nur funktioniert, solange eine Adresse angegeben ist.
Wenn keine Adresse in der Organisation angegeben ist, funktioniert nämlich mein Aufruf nicht mehr.
Beste Grüße
Tim

@barredterra barredterra changed the title feat: link contact_person(LAN-699) feat: link contact_person (LAN-699) Jul 18, 2023
@diePuppe
Copy link
Contributor Author

diePuppe commented Aug 8, 2023

Hallo @barredterra,
hab das jetzt mit Link und unlink umgesetzt, außerdem wird die Adresse automatisch aktualisiert, wenn die Adresse beim Member geändert wird.

Bildschirmaufzeichnung.vom.08.08.2023.12.16.15.webm

@barredterra
Copy link
Member

barredterra commented Aug 14, 2023

Closing in favor of 20a10d7

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