Skip to content

TicketArticleCreate modified table #12

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

Open
wants to merge 1 commit into
base: znuny-7_1
Choose a base branch
from

Conversation

MIPMHannes
Copy link
Contributor

Subject description fixed. From and To is only mandatory if CommunicationChannel is EMail (I guess?).
The fields To and From seem to be not mandatory if Communication Channel is Internal. I guess you only need those fields if CommunicationChannel is set to EMail?

Subject description fixed. From and To is only mandatory if CommunicationChannel  is EMail (I guess?)
@dignin dignin added the verified This issue or pull request was verified. label Mar 17, 2025
@dignin dignin self-assigned this Mar 17, 2025
@dignin dignin requested a review from rkaldung May 14, 2025 12:39
@rkaldung
Copy link
Member

TLDR: To and From and not mandatory

The From is needed for the CommunicationChannels Email, Phone, and Internal. If no From is provided, it is generated based on the parameter UserID (== the current user if this parameter is not used in the transition action).
The To field is optional, there is no check for its presence in the code and the database field is not mandatory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified This issue or pull request was verified.
Development

Successfully merging this pull request may close these issues.

3 participants