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

import file with calculated value from an external file that doesn't exist don't crash #2396

Merged
merged 3 commits into from
Oct 10, 2019

Conversation

Grummfy
Copy link
Contributor

@Grummfy Grummfy commented Oct 10, 2019

Description of the Change

fix for #2392

I didn't succeed to launch unit test locally, so I suppose it work ;)

@patrickbrouwers
Copy link
Member

Can you fix the code style? Also would be useful if you could give the PR a more descriptive name :) Thanks!

fix codestyle
@Grummfy Grummfy changed the title Fix 2392 import file with calculated value from an external file that doesn't exist don't crash Oct 10, 2019
@Grummfy
Copy link
Contributor Author

Grummfy commented Oct 10, 2019

here it is

@patrickbrouwers
Copy link
Member

Thanks! Looks good to me

@patrickbrouwers patrickbrouwers merged commit 575f824 into SpartnerNL:3.1 Oct 10, 2019
@Grummfy Grummfy deleted the fix-2392 branch October 10, 2019 13:12
@Grummfy
Copy link
Contributor Author

Grummfy commented Oct 10, 2019

thanks, that was fast!
Could you tag and create a new release?

@patrickbrouwers
Copy link
Member

Will release a new version in a few days, want to include a few more things before I release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants