Excel adapter fixed to allow for enum values

[pawelbaran]

NOTE: Depends on

BHoM/BHoM_Engine#3020

Issues addressed by this PR

Closes #14

Test files

Changelog

Additional comments

[pawelbaran]

@BHoMBot check core
@BHoMBot check compliance
@BHoMBot check null-handling
@BHoMBot check serialisation

[bhombot-ci]

@pawelbaran to confirm, the following actions are now queued:

• check core
• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance
• check null-handling
• check serialisation

There are 8 requests in the queue ahead of you.

[adecler]

For Enum, it is better to use IToText so it makes use of the DisplayText attribute.
Given that IToText needs to be improve to support primitives, I woudl suggest something like this for now: y => y is Enum ? y.IToText() : y.ToString() (to be tested)
Unless, you want to fix the IToText for primitives of course 😄

[pawelbaran]

@BHoMBot check core
@BHoMBot check compliance

[bhombot-ci]

@pawelbaran to confirm, the following actions are now queued:

• check core
• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance

There are 7 requests in the queue ahead of you.

[adecler]

Works well for me.
I have noticed some issue when goDeep is set to true but this is unrelated to this PR so I have raised a separate issue (#16) and will fix once this one is merged

[bhombot-ci]

@pawelbaran just to let you know, I have provided a check-versioning result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @pawelbaran on BHoM_Engine

[pawelbaran]

@BHoMBot check serialisation
@BHoMBot check null-handling

[bhombot-ci]

@pawelbaran to confirm, the following actions are now queued:

• check serialisation
• check null-handling

There are 14 requests in the queue ahead of you.

[bhombot-ci]

@pawelbaran just to let you know, I have provided a check-versioning result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @pawelbaran on BHoM_Engine

[bhombot-ci]

@pawelbaran just to let you know, I have provided a check-installer result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @pawelbaran on BHoM_Engine

[bhombot-ci]

@pawelbaran just to let you know, I have provided a check-ready-to-merge result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @FraserGreenroyd on BHoM_Engine