kartoza / django-bims

https://testing.healthyrivers.kartoza.com
GNU Affero General Public License v3.0
12 stars 19 forks source link

Location site - update site code not working for > 1 site at a time #3172

Closed helendallas closed 2 years ago

helendallas commented 2 years ago

We need to be able to update the Site code in the Location Site table.

At the moment it only works if you select one site at a time. This will take ages.

Please can you see why it does not work for more - either 100 at a time, or to be able to select all

image

https://rbis.ur.ac.rw/admin/bims/locationsite/?q=&o=2

This is also an issue in RBIS.

dimasciput commented 2 years ago

@Fanevanjanahary, I think it's better to run the task in the background if there are more than 10 sites.

helendallas commented 2 years ago

@dimasciput @Fanevanjanahary Please can you prioritise this. Many sites that are already on FBIS do not have rivername or geomorphological zone. In addition to updating site code in bulk, we need to be able to update

image

I don't think this is working correctly. please can you check.

helendallas commented 2 years ago

@Fanevanjanahary Please update me on where I can test this please

fanevanjanahary commented 2 years ago

@helendallas this is fixed by #3205, waiting for @dimasciput to deploy it

helendallas commented 2 years ago

@dimasciput Please update me on this ticket?

dimasciput commented 2 years ago

@helendallas you can try it in live site

helendallas commented 2 years ago

Still No update site codes to run in background?

Works for 1 to 5, times out when I select 10

image

Updating location context for sites in background

Very slow, even for one site, doesnt seem to be working

image

dimasciput commented 2 years ago

@helendallas

Works for 1 to 5, times out when I select 10

If you select more than 10 sites, they will be processed in the background automatically.

helendallas commented 2 years ago

Thanks, this seems to be working now. There are still some sites that don't have the correct code but that is because they are no where near a river? or out of SA

Closing ticket