Converting SP dates to readable excel date format

[Rundez]

Your checklist for this pull request

• Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your main!
• Make sure you are making a pull request against the dev branch (left side). Also you should start your branch off dev.
• Check the commit's or even all commits' message
• Check if your code additions will fail linting checks
• Remember: Add PR description to CHANGELOG with the ID that matches this PR

Description

When exporting project information with dates from "Porteføljeoversikt", the dates were not recognized by excel due to the format.
The solution: Convert the Sharepoint date format to a standard JS date object recognized by excel.

How to test

• Navigate to "Porteføljeoversikt", add start and end date columns, then export to excel.
• The dates should now be recognized by excel.

Relevant issues (if applicable)

fixes #487

💔Thank you!

[olemp]

Too hardcoded in my opinion. Only works for these two date fields. We should look into a more dynamic solution that works for all date fields, including custom ones.

[tarjeieo]

Too hardcoded in my opinion. Only works for these two date fields. We should look into a more dynamic solution that works for all date fields, including custom ones.

I agree that it's not ideal. How about iterating over properties of the item, converting those ending with *OWSDATE to proper date values?

[olemp]

Too hardcoded in my opinion. Only works for these two date fields. We should look into a more dynamic solution that works for all date fields, including custom ones.

I agree that it's not ideal. How about iterating over properties of the item, converting those ending with *OWSDATE to proper date values?

We should have access to the columns array, where we have the column config and can catch if it's a date.

[Rundez]

I agree. Added a solution that iterates the columns and checks the field type. This should also convert any custom date values.

[tarjeieo]

Any input here @olemp ? Can this be merged?

[olemp]

Any input here @olemp ? Can this be merged?

Still want to have a look through it. Should have time this week - tomorrow or Thu.

[tarjeieo]

Any input here @olemp ? Can this be merged?

Still want to have a look through it. Should have time this week - tomorrow or Thu.

Friendly reminder :)

[olemp]

Se comments.

[olemp]

@Rundez You're using Array.map without returning anything. Seems like it's Array.forEach you want to use here.

[olemp]

See Array.map documentation

[Rundez]

@olemp Changed map to foreach.....

[danielwiese]

Tested OK