Repository navigation
feat(nowe_raporty): nowy raport w adminie zakłada definicję + pozycja w menu - #880
Merged
Merged
Conversation
… w menu Dodanie raportu flexible-reports w adminie (formularz albo „Klonuj") zakłada mu nieaktywną DefinicjaRaportu (poziom uczelnia, dostęp zalogowani, slug z przyrostkiem przy kolizji). Bez tego nowy Report nie trafiał do menu „raporty" na stronie, a redaktor musiał pamiętać o drugim kroku w innym miejscu panelu. Hak siedzi w ReportAdmin.log_addition, nie w post_save: seeding i kod tworzący Report programowo same zakładają definicje z właściwymi uprawnieniami — sygnał nadpisałby im slug/uprawnienia. „Definicje raportów" są pierwszą pozycją sekcji „Raporty" w menu admina. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Po co
Dodanie raportu we flexible-reports nie powodowało, że pojawiał się w menu „raporty” na stronie. Trzeba było jeszcze ręcznie założyć definicję raportu. Ta z kolei nie miała pozycji w menu admina i dało się do niej wejść tylko z listy aplikacji na dashboardzie.
Zmiany
Menu admina → Raporty: pierwsza pozycja to teraz „Definicje raportów”. Tam ustawiasz, co pokazuje menu „raporty” na stronie, komu i na których uczelniach. Pozostałe pozycje (Raporty / Źródła danych / Tabele) to „silnik” flexible-reports.
Auto-definicja: dodanie raportu w adminie (formularz albo „Klonuj”) zakłada mu definicję:
Reportnie mówi, jakiego poziomu dotyczy,-2,-3…Redaktor ustawia poziom, dostęp i uczelnie, zaznacza „aktywny” i raport pojawia się w menu.
Dlaczego
log_addition, a niepost_saveBppReportAdmin(podmieniaReportAdminz flexible-reports) wywołujeutworz_definicje_dla()wlog_addition, bo przechodzą przez nią oba sposoby dodania raportu w adminie: add view (już po zapisaniu inline'ów) iclone_view. Sygnałpost_saveodpalałby się też przy seedingu. Definicja powstawałaby wtedy przed definicją seedingu, a seeding, który pomija istniejące slugi, zostawiłby np. raport autorów z poziomem uczelnia i bez dostępu „wszyscy”. Kolidowałby też z testami tworzącymiReport+ definicję z tym samym slugiem. Kod tworzącyReportprogramowo nadal sam decyduje o definicji, co pilnuje test.Testy
src/nowe_raporty/tests/test_auto_definicja.py: dodanie przez admin, edycja (bez duplikatu), kolizja sluga, długi tytuł/slug, klonowanie, brak definicji dlaReporttworzonego poza adminem. Przed implementacją oglądane na czerwono.src/django_bpp/tests/test_menu.py: pierwsza pozycja „Raporty” to definicje.Lokalnie
src/nowe_raporty+src/django_bpp: 225 passed.pre-commitczysto.🤖 Generated with Claude Code