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.

Sign up for GitHub

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

Jump to bottom

DO NOT MERGE - Thunder 8 #818

Open
wants to merge 19 commits into
base: 7.3.x
Choose a base branch
from
Open

DO NOT MERGE - Thunder 8 #818

wants to merge 19 commits into from

Conversation

chrfritsch
Copy link
Member

Make sure these boxes are checked before submitting your pull request - thank you!

  • All coding styles are fulfilled. (How to check for cs issues?)
  • All tests are running locally. (How to run the test?)
  • Necessary update hooks are provided.
  • User roles have correct access for new introduced permission.
  • Every thunder module has a README.md in its root. Follow this guidelines, but we don't need every topic.
  • Code is covered with well-balanced amount of inline comments.
  • New features or changes are documented.

If you are really awesome, then your feature is covered by additional tests. Well done!

@chrfritsch chrfritsch added the DO NOT MERGE It's prohibited to merge this branch label Jul 11, 2024
@chrfritsch
Remove thunder_demo module
064200b 
* Remove thunder_demo module

* Update docs

---------

Co-authored-by: Christian Fritsch <[email protected]>
@ol0lll @chrfritsch
Remove blazy and slick integration
dc68aff 
* Remove and cex

* add update hook

* remove deps

* add back in composer.json

* fix filter full_html deps

* fix tests

* Fix tests

* fix tests 2

* Remove from composer.json

* empty

* Remove leftovers

---------

Co-authored-by: Christian Fritsch <[email protected]>
Co-authored-by: Christian Fritsch <[email protected]>
@chrfritsch
Remove jquery_ui (#820)
362cf2d 
* Remove jquery_ui

* Remove

* Docs

---------

Co-authored-by: Christian Fritsch <[email protected]>
@chrfritsch
Remove nightwatch tests
f49a604 
Co-authored-by: Christian Fritsch <[email protected]>
@chrfritsch
Add strict typing everywhere, use php attributes and property promotion
b58b038 
* Add strict typing everywhere, use php attributes and property promotion

* Fix

* Moar in tests

* Rename fieldContext

---------

Co-authored-by: Christian Fritsch <[email protected]>
@chrfritsch
Remove responsive_preview module
3f731a5 
* Remove responsive_preview module

* Remove baseline entry

* Docs

* Update test.yml

---------

Co-authored-by: Christian Fritsch <[email protected]>
@dbosen @chrfritsch
Use ief 3.0
a15ff6c 
* allow ief 3.0

* Update composer.json

---------

Co-authored-by: Christian Fritsch <[email protected]>
@ol0lll ol0lll temporarily deployed to thunder-pr-818 July 15, 2024 13:07 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
DO NOT MERGE It's prohibited to merge this branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chrfritsch @ol0lll @dbosen