inbo / reporting-rshiny-grofwildjacht

Rshiny app for grofwildjacht
https://grofwildjacht.inbo.be/
MIT License
1 stars 1 forks source link

[BUG] WBE renamed => no longer selectable #405

Closed SanderDevisscher closed 1 year ago

SanderDevisscher commented 1 year ago

Describe the bug I've renamed WBE Zuid-Vlaamse Valleien Regio 1 (wbenr: 247 & kbonr: 455431529) into Hemelrijk but when I try to select it from the list (after login as admin) it is no longer available.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'private page as admin'
  2. Click on 'list'
  3. Scroll down to 'He'
  4. See absence of Hemelrijk

Expected behavior Hemelrijk should be available for selection.

Additional context WBE_nr & kbonr remain unchanged. Zuid-Vlaamse Valleien Regio 1 is not available as well.

SanderDevisscher commented 1 year ago

I checked and the wbe is present & correct in both .csv files on the UAT bucket (Data_Partij_Cleaned & Data_Partij_Cleaned_processed)

mvarewyck commented 1 year ago

@SanderDevisscher Did you check in docker with the kbo = 'admin' argument explicitly? It looks like 'Hemelrijk' is in the list then for me.

If you don't use the argument, there is a default subset of KBO's shown in the list which doesn't contain 'Hemelrijk'.

docker run -p 3001:3838 -it inbo/grofwild R -e "Sys.setenv('AWS_DEFAULT_REGION'='eu-west-1','AWS_ACCESS_KEY_ID'='XXX,'AWS_SECRET_ACCESS_KEY'='XXX'); reportingGrofwild::runWildApp(public=FALSE, kbo ='admin')"
SanderDevisscher commented 1 year ago

What should be done to include 'Hemelrijk' in the default list ?

mvarewyck commented 1 year ago

What should be done to include 'Hemelrijk' in the default list ?

Then I have to add its KBO to the list here I can change the default to anything: Would you prefer to have the default to be 'admin'?

SanderDevisscher commented 1 year ago

I'll rebuild the app using the admin credentials

SanderDevisscher commented 1 year ago

Hemelrijk is displayed as it should