You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
freue mich, heute meinen allerersten PR überhaupt einzureichen. Es hat Spaß gemacht, mich in das Thema reinzufuchsen und ich bin ein klein wenig stolz, jetzt auch mal aktiv etwas zum Projekt beitragen zu können! 😊
Was wurde gemacht?
Frontend: In der Twig-Übersicht (templates_workspace.html.twig) einen neuen Schalter (Form-Switch) inklusive Bootstrap-Tooltip hinzugefügt.
Backend: Die Entity Template um das Feld hidden erweitert, Getter/Setter eingebaut und das TemplateRepository sowie den TemplatesService angepasst, damit versteckte Vorlagen standardmäßig herausgefiltert werden.
Hi @MeisterAdebar, cool danke dir 😎
Habs mir angesehen und kann erst mal keine negativen Seiteneffekte feststellen. Sieht gut aus.
Du müsstest aber noch eine DB Migrationsdatei anlegen, damit das neue hidden Feld auch autom. in der DB eingefügt wird. Die vorhandenen liegen alle in migrations/
Wenn du ganz gut bist lässt du am Ende auch noch das test-Script laufen, was die Anwendung einmal in einer Test-DB komplett initialisiert und alle vorhandenen Tests laufen lässt. Dann siehst du auch, ob dein Migrationsscript sauber durchläuft.
Anleitung siehe hier: https://github.com/developeregrem/fewohbee/wiki/Development#automated-test-run
pi@raspi3b:/var/www/html/fewohbee $ sudo bin/run-tests.sh
==> Resetting database for env 'test'
→ php bin/console doctrine:database:drop --force --if-exists --no-interaction
Database `fewohbee_test_suite_test` for connection named default doesn't exist. Skipped.
→ php bin/console doctrine:database:create --if-not-exists --no-interaction
Created database `fewohbee_test_suite_test` for connection named default
→ php bin/console doctrine:migrations:migrate --no-interaction
[notice] Migrating up to DoctrineMigrations\Version20260526115836
[warning] Migration DoctrineMigrations\Version20190906160425 was executed but did not result in any SQL statements.
[warning] Migration DoctrineMigrations\Version20190915110513 was executed but did not result in any SQL statements.
[notice] finished in 24888.7ms, used 28M memory, 41 migrations executed, 296 sql queries
[OK] Successfully migrated to version: DoctrineMigrations\Version20260526115836
→ php bin/console app:first-run --username=test-admin --password=ChangeMe123! --first-name=Test --last-name=Admin --email=test-admin@example.com --accommodation-name=FewohBee Test Suite --load-sample-data --no-interaction
This process will ask you questions in order to prepare the app for its first use.
==================================================================================
! [NOTE] User and Roles created.
! [NOTE] Templates prepared.
! [NOTE] Accommodation prepared.
! [NOTE] Customers prepared.
! [NOTE] Base templates loaded.
! [NOTE] Sample data loaded.
! [NOTE] Example workflows created.
! [NOTE] Default guest categories prepared.
[OK] All done! You can now navigate to the app and login with the provided username and password.
PHPUnit 12.5.26 by Sebastian Bergmann and contributors.
Runtime: PHP 8.4.21
Configuration: /var/www/html/fewohbee/phpunit.xml.dist
..............................S................................ 63 / 637 ( 9%)
............................................................... 126 / 637 ( 19%)
............................................................... 189 / 637 ( 29%)
............................................................... 252 / 637 ( 39%)
............................................................... 315 / 637 ( 49%)
............................................................... 378 / 637 ( 59%)
............................................................... 441 / 637 ( 69%)
............................................................... 504 / 637 ( 79%)
............................................................... 567 / 637 ( 89%)
............................................................... 630 / 637 ( 98%)
....... 637 / 637 (100%)
Time: 00:50.411, Memory: 129.00 MB
OK, but some tests were skipped!
Tests: 637, Assertions: 2049, Skipped: 1.
ich glaube du musst den commit bei dir nochmal in einen neuen branch rüberziehen und für dieses Thema einen neuen PR erstellen. Sonst kann ich es nicht mergen :(
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Mein allererster Pull Request! 🎉
Hi,
freue mich, heute meinen allerersten PR überhaupt einzureichen. Es hat Spaß gemacht, mich in das Thema reinzufuchsen und ich bin ein klein wenig stolz, jetzt auch mal aktiv etwas zum Projekt beitragen zu können! 😊
Was wurde gemacht?
templates_workspace.html.twig) einen neuen Schalter (Form-Switch) inklusive Bootstrap-Tooltip hinzugefügt.Templateum das Feldhiddenerweitert, Getter/Setter eingebaut und dasTemplateRepositorysowie denTemplatesServiceangepasst, damit versteckte Vorlagen standardmäßig herausgefiltert werden.