Skip to content

Eingebettete Vorlagen in der Vorlagenauswahl ausblenden#233

Open
MeisterAdebar wants to merge 2 commits into
developeregrem:masterfrom
MeisterAdebar:master
Open

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

Conversation

@MeisterAdebar
Copy link
Copy Markdown

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
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 3 times, most recently from 84ccd07 to c225397 Compare May 29, 2026 15:10
feat: add A5 page format support for PDF templates
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