Skip to content
This repository has been archived by the owner on Mar 31, 2021. It is now read-only.

Fix 23h for energinet and flat polyfill. #435

Merged
merged 3 commits into from
Aug 17, 2020
Merged

Conversation

pierresegonne
Copy link
Member

@pierresegonne pierresegonne commented Aug 17, 2020

@pierresegonne pierresegonne added bug Something isn't working 🔌 electricity meter labels Aug 17, 2020
@corradio corradio requested review from madsnedergaard and removed request for corradio August 17, 2020 07:06
@corradio
Copy link
Member

@madsnedergaard
Copy link
Member

Is the "flatten" part supposed to be committed? Doesn't seem like it's used anywhere, right?

@pierresegonne
Copy link
Member Author

Found a work around (remembered about the beautiful existence of lodash!) I am currently testing it

The flatten part is a polyfill, .flat is still experimental in JS so sometimes it works, sometimes it does not

@pierresegonne
Copy link
Member Author

@madsnedergaard this will need to be merged before running the integration on production for bloom

Copy link
Member

@madsnedergaard madsnedergaard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@pierresegonne pierresegonne merged commit 3158fbd into master Aug 17, 2020
@pierresegonne pierresegonne deleted the ps/improve-energinet branch August 17, 2020 09:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working 🔌 electricity meter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants