Skip to content

Release 6.0.1#169

Merged
jolelievre merged 10 commits intomasterfrom
dev
May 12, 2023
Merged

Release 6.0.1#169
jolelievre merged 10 commits intomasterfrom
dev

Conversation

@Progi1984
Copy link
Member

@Progi1984 Progi1984 commented May 9, 2023

Questions Answers
Description? Release 6.0.1
Type? improvement
BC breaks? no
Deprecations? no
Fixed ticket? N/A
How to test? Release Test (Module compatible >= 8.1)

@Progi1984 Progi1984 added this to the 6.0.1 milestone May 9, 2023
@Progi1984 Progi1984 requested review from a team May 9, 2023 07:44
@AureRita
Copy link

AureRita commented May 9, 2023

Hi @Progi1984

Thank you for your PR, I tested it and

On 8.0.3
when I installed directly from the version 6.0.1 I've a red alert that it cannot instal the module
and
When I do the upgrade from an installed version, I've an exception if i'm on debug mode (if I'm not in debug mode, I'll have an 500 error when I clic on the button "+ New block"
You can see these issue on this video :

Untitled_.May.9.2023.5_30.PM.webm

On 1.7.8.9 :
I think we have the same exception of the version 8.0.3 but this one is showed even if you're not in link list ( when debug mode is off)
image

Thank you

@AureRita AureRita added Waiting for author Waiting for author feedback and removed Waiting for QA labels May 9, 2023
@Progi1984
Copy link
Member Author

@AureRita The version 6.0 is only compatible with PS 8.1+. Please check again.

@Progi1984 Progi1984 added Waiting for QA and removed Waiting for author Waiting for author feedback labels May 9, 2023
@paulnoelcholot paulnoelcholot self-assigned this May 12, 2023
@paulnoelcholot
Copy link

Hello @Progi1984,

I tested your PR and it's GOOD for me! 🎉

Thanks!

@paulnoelcholot paulnoelcholot added QA ✔️ Status: Check done, Code approved and removed Waiting for QA labels May 12, 2023
@paulnoelcholot paulnoelcholot removed their assignment May 12, 2023
@jolelievre jolelievre merged commit 238cc92 into master May 12, 2023
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.

9 participants