-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[api] Add range of amounts (min, max) for feature per PU [MRXN23-444] #1551
[api] Add range of amounts (min, max) for feature per PU [MRXN23-444] #1551
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
b5ca654
to
9f2dcc1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @yulia-bel! just some performance notes to make sure things work smoothly irrespective(ish) of how many puvspr_records
are in a given instance, for how many distinct feature_id
s, and of how many features we want to set min/max for at once (through the new GeoFeaturesService.saveAmountRangeForFeatures()
method)
api/apps/api/src/modules/geo-features/import/features-amounts-upload.service.ts
Show resolved
Hide resolved
0f1db02
to
7172b6a
Compare
Add range of amounts (min, max) for feature
Overview
This PR adds 2 new columns to (apiDB).features table to store min and and amount values.
As agreed before, service (in GeoFeaturesServices) will retrieve values from (geoDB)puvspr_calculations table by featureId and save them to (apiDB).features table. the serviced will be reused for all feature types..
Tests updated accordingly and to be reviewed when implementing https://vizzuality.atlassian.net/browse/MRXN23-460 (currently null values are being saved)
Designs
Link to the related design prototypes (if applicable)
Testing instructions
Please explain how to test the PR: ID of a dataset, steps to reach the feature, etc.
Feature relevant tickets
Link to the related task manager tickets
Checklist before submitting
develop
.deploying to staging/production, please add brief testing instructions
to the deploy checklist (
docs/deployment-checklist.md
)