Skip to content

Conversation

@KaMeHb-UA
Copy link
Contributor

@KaMeHb-UA KaMeHb-UA commented Sep 27, 2022

+ docs =)

P.S. I hadn't enough time to test all the stuff properly so there may be some small issues

@KaMeHb-UA KaMeHb-UA changed the title upgrade module to work with nodegui 0.57.1 upgrade module to work with latest nodegui (0.57.1) Sep 27, 2022
@KaMeHb-UA
Copy link
Contributor Author

tested a bit, there is issues with layout, temporary marking this PR as draft

@KaMeHb-UA KaMeHb-UA marked this pull request as draft September 27, 2022 00:33
@KaMeHb-UA KaMeHb-UA marked this pull request as ready for review September 27, 2022 06:07
@KaMeHb-UA
Copy link
Contributor Author

fixed basic issues, sample app operates normally but there is still need to check if all the stuff's working well by someone else

Copy link
Collaborator

@a7ul a7ul left a comment

Choose a reason for hiding this comment

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

Tested this and it looks amazing.
Thanks for the great work

@a7ul a7ul merged commit 742540a into nodegui:master Oct 4, 2022
@a7ul
Copy link
Collaborator

a7ul commented Oct 4, 2022

@allcontributors add @KaMeHb-UA for code

@allcontributors
Copy link
Contributor

@a7ul

I've put up a pull request to add @KaMeHb-UA! 🎉

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.

2 participants