PolicyEngine / policyengine-api

PolicyEngine's REST API for computing policy impacts.
GNU Affero General Public License v3.0
7 stars 19 forks source link

Change how queue positioning is reported #1579

Closed anth-volk closed 1 week ago

anth-volk commented 1 week ago

Fixes #1578. This PR adds a separate queue_position element to the output in cases where the API is in the process of calculating and ensures that both display the queue position as "0" instead of "None" when an item is being actively calculated.

This is necessary to fix https://github.com/PolicyEngine/policyengine-app/issues/1883.

anth-volk commented 1 week ago

Alternatively, the message could be removed and we could maintain only the queue_position, but may be beneficial for external API users, if any

MaxGhenis commented 1 week ago

I'd lean to removing the message

anth-volk commented 1 week ago

Will open a separate issue to do so