BHoM / BHoM/Excel_UI

Change error type returned to an actual error type

Open
#260 0 comments 1 reaction 1 assignee Claimed by @adecler View on GitHub
type:feature
Dominant language
C#
Stars
5
Forks
5
PR merge metrics
No merged PRs in 30d

Description

#### Description:

When something crashes in code executed, might be due to anything really, you generally get `ExcelErrorNull` returned from the toolkit. This type of error is _not_ recognised as an error by the native `=IFERROR(` command in Excel.

This can be quite annoying when you want to set up a sheet to be ready to receive data from an adapter or similar, where a simple `=IFERROR( do some stuff, "")` can be a quite nice way to keep the sheet looking clean without to much hassle.

Would suggest a `#VALUE` or similar to be returned instead (not sure if there is like `#NULL' or similar that would be an even better fit), basically, something picked up by Excel as being an error.

https://github.com/BHoM/Excel_Toolkit/blob/master/Excel_UI/Addin/AddIn_Converts.cs seem to be the main file handling this from my quick search! :)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.