500 restyling

[mvarewyck]

As suggested during our last meeting: PR with most urgent fixes #507, #509, #510, #523

[SanderDevisscher]

#525 has priority

[SanderDevisscher]

The deployment succeded
@soriadelva can you test faunabeheer-uat.inbo.be to check if you can find any bugs ?
Ifnot please approve this PR.
I'll do the same!

[SanderDevisscher]

Just a list to check if they were fixed:

• Tabel Kencijfers table is missing #507
• [BUG] A timeout occurs when going to subcategories immediately or when only selecting a species but not a category #509
• Always provide all categories, subcategories and graphs on the upper toolbar #510
• [BUG] disable 'Inschatting Kosten' #523

[SanderDevisscher]

All the issues listed as "dringend" and some extras were fixed.

[mvarewyck]

• De kaart van het toekomstig verspreidingsgebied laat steeds dat van Wild zwijn zien ongeacht welke soort ik kies. Dit zou data driven moeten zijn. Komt dit omdat ook hier "wildsoort" in de data ontbreekt ?

@SanderDevisscher Er is idd geen indicatie voor wildsoort in de Pixels_ModelOutput_toekomst_verspr en Municipalities_ModelOutput_toekomst_verspr shape files. Kunnen jullie dit toevoegen? In tussentijd zal ik een temporary fix in de code doen.

[SanderDevisscher]

• De kaart van het toekomstig verspreidingsgebied laat steeds dat van Wild zwijn zien ongeacht welke soort ik kies. Dit zou data driven moeten zijn. Komt dit omdat ook hier "wildsoort" in de data ontbreekt ?

@SanderDevisscher Er is idd geen indicatie voor wildsoort in de Pixels_ModelOutput_toekomst_verspr en Municipalities_ModelOutput_toekomst_verspr shape files. Kunnen jullie dit toevoegen? In tussentijd zal ik een temporary fix in de code doen.

Ik zal bekijken om een kolom wildsoort toe te voegen.

[SanderDevisscher]

this should be fixed as well. PS aantal drukjachten voor wild zwijn

[mvarewyck]

this should be fixed as well. PS aantal drukjachten voor wild zwijn

I cannot reproduce this. Is this via direct routing Wild zwijn > Aantal drukjachten ?

[SanderDevisscher]

this should be fixed as well. PS aantal drukjachten voor wild zwijn

I cannot reproduce this. Is this via direct routing Wild zwijn > Aantal drukjachten ?

It exists on the faunabeheer-uat.inbo.be when you go via the normal route Wild zwijn > beheer > afschot per jachtmethode > Aantal drukjachten as well as when you go directly to Aantal drukjachten & select wild zwijn. Selecting a region other than flanders shows a graph but the year slider remains broken.

[mvarewyck]

this should be fixed as well. PS aantal drukjachten voor wild zwijn

I cannot reproduce this. Is this via direct routing Wild zwijn > Aantal drukjachten ?

It exists on the faunabeheer-uat.inbo.be when you go via the normal route Wild zwijn > beheer > afschot per jachtmethode > Aantal drukjachten as well as when you go directly to Aantal drukjachten & select wild zwijn. Selecting a region other than flanders shows a graph but the year slider remains broken.

It was due to NA values for 'jachtmethode_comp' introduced in the latest data rshiny_reporting_data_ecology_processed.parquet. This is now handled correctly by the code.

In case you don't expect this, you can extract these records:

ecoData <- loadRawData(type = "eco")
ecoData[is.na(ecoData$jachtmethode_comp), ]
# A tibble: 33 × 27
   ID                wildsoort afschotjaar afschot_datum provincie geslacht_comp
   <chr>             <chr>           <int> <date>        <fct>     <fct>        
 1 bca46d4637693f63… Wild zwi…        2024 2025-01-10    Antwerpen Vrouwelijk   
 2 167860f817e7fde3… Wild zwi…        2024 2025-01-10    Antwerpen Vrouwelijk   
 3 0cee452506a1df2a… Wild zwi…        2024 2025-01-10    Antwerpen Vrouwelijk   
 4 3b05fd3b7e3970d9… Wild zwi…        2024 2025-01-10    Antwerpen Vrouwelijk   
 5 8f474e4c07a2a698… Wild zwi…        2024 2025-01-10    Antwerpen Vrouwelijk   
 6 c334271a8e34bf3e… Wild zwi…        2024 2025-01-10    Antwerpen Mannelijk    
 7 4fc934b4a522e9bf… Wild zwi…        2024 2025-01-10    Antwerpen Vrouwelijk   
 8 ac0f3cf3d491352e… Wild zwi…        2024 2025-01-10    Antwerpen Mannelijk    
 9 d95d70ad793adb8e… Wild zwi…        2024 2025-01-10    Antwerpen Mannelijk    
10 edab521b930e95b1… Wild zwi…        2024 2025-01-09    Vlaams B… Vrouwelijk   
# ℹ 23 more rows

[SanderDevisscher]

@mvarewyck geef je een seintje als ik mag redeployen ?

[mvarewyck]

@mvarewyck geef je een seintje als ik mag redeployen ?

Ik heb nog wat verder getest op navigatie. De app crasht nu niet langer in onderstaand scenario, maar navigatie is nog niet optimaal. Ik denk dat we navigatie best grondiger herzien later ifv #538 #504

Select Wild zwijn --> beheer, then select Haas in the left menu, then select Verspreiding in the top menu, see the error.

@SanderDevisscher Klaar om te deployen.

[SanderDevisscher]

I did some testing and I did not encounter any more breaking issues for me this is a GO! 🥳

However you correctly pointed out we should review navigation but this can wait untill after launch.

stale