Skip to content

Configure typescript properly to fix the build#160

Merged
NeOMakinG merged 1 commit intoPrestaShop:devfrom
NeOMakinG:fix-build-ts
Nov 17, 2022
Merged

Configure typescript properly to fix the build#160
NeOMakinG merged 1 commit intoPrestaShop:devfrom
NeOMakinG:fix-build-ts

Conversation

@NeOMakinG
Copy link

Questions Answers
Description? Build wasn't working properly, probably since some core changes on develop and 8.0.x
Type? bug fix
BC breaks? no
Deprecations? no
How to test? npm i && npm run build inside views

Copy link

@MhiriFaten MhiriFaten left a comment

Choose a reason for hiding this comment

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

Hello @NeOMakinG ,

I still find the issue and I'm still investigating.
Also, there are some checks were not successful

image

Could you check please?
Thank you.

@MhiriFaten MhiriFaten added Waiting for author Waiting for author feedback and removed Waiting for QA labels Nov 16, 2022
@MhiriFaten MhiriFaten removed their assignment Nov 16, 2022
@NeOMakinG
Copy link
Author

Hello @NeOMakinG ,

I still find the issue and I'm still investigating. Also, there are some checks were not successful

image

Could you check please? Thank you.

It's not related to that PR at all

@NeOMakinG NeOMakinG added Waiting for QA and removed Waiting for author Waiting for author feedback labels Nov 17, 2022
@MhiriFaten MhiriFaten self-assigned this Nov 17, 2022
Copy link

@MhiriFaten MhiriFaten left a comment

Choose a reason for hiding this comment

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

Hello @NeOMakinG ,

The build works properly 🎉

image

It's QA ✔️

Thank you!

@MhiriFaten MhiriFaten added QA ✔️ Status: Check done, Code approved and removed Waiting for QA labels Nov 17, 2022
@NeOMakinG NeOMakinG added this to the 5.0.6 milestone Nov 17, 2022
@NeOMakinG NeOMakinG merged commit 0d42593 into PrestaShop:dev Nov 17, 2022
@NeOMakinG
Copy link
Author

Thanks @MhiriFaten

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

QA ✔️ Status: Check done, Code approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants