Skip to content

Eingebettete Vorlagen in der Vorlagenauswahl ausblenden#233

Closed
MeisterAdebar wants to merge 0 commit into
developeregrem:masterfrom
MeisterAdebar:master
Closed

Eingebettete Vorlagen in der Vorlagenauswahl ausblenden#233
MeisterAdebar wants to merge 0 commit into
developeregrem:masterfrom
MeisterAdebar:master

Conversation

@MeisterAdebar

Copy link
Copy Markdown
Contributor

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?

  • 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.

@developeregrem

Copy link
Copy Markdown
Owner

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

@MeisterAdebar

Copy link
Copy Markdown
Contributor Author
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.

@developeregrem developeregrem added this to the 4.9.0 milestone May 27, 2026
@MeisterAdebar MeisterAdebar force-pushed the master branch 4 times, most recently from 6ac9a87 to 6d50ddc Compare May 29, 2026 17:03
@developeregrem

Copy link
Copy Markdown
Owner

Wieso ist der geschlossen?

@MeisterAdebar

Copy link
Copy Markdown
Contributor Author

Hab keine Ahnung wie das passiert. Bewusst hab ich ihn nicht geschlossen.

@developeregrem

Copy link
Copy Markdown
Owner

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 :(

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