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

Make list access private #372

Open
stephanstapel opened this issue Oct 2, 2024 · 3 comments
Open

Make list access private #372

stephanstapel opened this issue Oct 2, 2024 · 3 comments

Comments

@stephanstapel
Copy link
Owner

stephanstapel commented Oct 2, 2024

  • Taxes
  • ServiceCharges
  • TradeLineItems
  • ReceivableSpecifiedTradeAccountingAccounts
  • CreditorBankAccounts
  • DebitorBankAccounts
@tobitege
Copy link
Contributor

tobitege commented Jan 5, 2025

Would you mind to clarify if you meant only write list access?
Such changes could have huge (breaking) consequences for any 3rd party software which uses them for their own purposes (special processing/validation/cleanup).

@stephanstapel
Copy link
Owner Author

I mean things like:

public List<AdditionalReferencedDocument> AdditionalReferencedDocuments { get; internal set; } = new List<AdditionalReferencedDocument>();

and

public void AddAdditionalReferencedDocument(string id, AdditionalReferencedDocumentTypeCode typeCode, DateTime? issueDateTime = null, string name = null, ReferenceTypeCodes referenceTypeCode = ReferenceTypeCodes.Unknown, byte[] attachmentBinaryObject = null, string filename = null)

I am aware of the impact, that's why I'm using semantic versioning for the component.

@tobitege
Copy link
Contributor

tobitege commented Jan 5, 2025

That's what I thought, thanks for the clarification. 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants