Skip to content

Use IBootstrap for the app bootstrap - #924

Merged
juliusknorr merged 2 commits into
masterfrom
feature/noid/use-ibootstrap-for-application
Jul 10, 2020
Merged

Use IBootstrap for the app bootstrap#924
juliusknorr merged 2 commits into
masterfrom
feature/noid/use-ibootstrap-for-application

Conversation

@MorrisJobke

Copy link
Copy Markdown
Member

I tested it and it seemed to work, but please have another look.

Also @ChristophWurst must be proud of this :)

Comment threadlib/AppInfo/Application.php Outdated
@MorrisJobke
MorrisJobkeforce-pushed the feature/noid/use-ibootstrap-for-application branch 3 times, most recently from 6c0e34f to ca2ce1cCompareJuly 9, 2020 18:18
@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

Ready for review and it feels so much better than before :)

@ChristophWurstChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Best thing that has happened in my life so far

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@juliusknorr
juliusknorrforce-pushed the feature/noid/use-ibootstrap-for-application branch from ca2ce1c to 759d679CompareJuly 10, 2020 05:58
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr

Copy link
Copy Markdown
Member

Hm cypress fails seem random, and not really related. Let's get this in 🚀

@juliusknorr
juliusknorr merged commit 44f7b64 into masterJul 10, 2020
@juliusknorr
juliusknorr deleted the feature/noid/use-ibootstrap-for-application branch July 10, 2020 09:00
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@MorrisJobke@juliusknorr@rullzer@ChristophWurst