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

[Feature Request] CompositeCell.Data compatibility issues with existing logic #417

Closed
Freymaurer opened this issue Jul 15, 2024 · 1 comment
Assignees

Comments

@Freymaurer
Copy link
Collaborator

Issue

To interact between ARCtrl and Excel for example i use the CompositeCell.GetContent() function. This returns 1, 3 or 4 strings in an array. Which i also use to reverse the information into the correct cell type.

The new Data cell also returns 3 strings and makes the logic not unique anymore. Combined with a lot more settable fields i am unsure how to progress from this point:

image

| Data d -> [| defaultArg d.Name ""; defaultArg d.Format ""; defaultArg d.SelectorFormat ""|]

Requirements

  • CompositeCell.GetContent must return an array of strings, which must have a unique number of strings according to CompositeCell type.
  • CompositeCell.GetContent must be fully reversible.
  • Due to the same issue, Data cell information must always fully be written in excel. Currently it omits complete empty columns.
@github-actions github-actions bot added the Status: Needs Triage This item is up for investigation. label Jul 15, 2024
@HLWeil HLWeil self-assigned this Aug 5, 2024
@HLWeil HLWeil removed the Status: Needs Triage This item is up for investigation. label Aug 5, 2024
@HLWeil
Copy link
Member

HLWeil commented Aug 6, 2024

After discussion with @Freymaurer, this is not possible.

Header is necessary to distinguish between the types of 3 cell Data content and e.g. Parameter content.

@HLWeil HLWeil closed this as completed Aug 6, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants