fix planning area assignment #352
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). marxan – ./app🔍 Inspect: https://vercel.com/vizzuality1/marxan/3oQGQ2UVzmpUUJnM1wJxVxJqcteZ marxan-storybook – ./app🔍 Inspect: https://vercel.com/vizzuality1/marxan-storybook/FaFchfpQQxvN4dpXk5SNeGcNAvPd |
| @@ -136,6 +136,7 @@ export class ProjectsCrudService extends AppBaseService< | |||
| */ | |||
| const project = await super.setDataCreate(create, info); | |||
| project.createdBy = info?.authenticatedUser?.id!; | |||
| project.planningAreaGeometryId = create.planningAreaId; | |||
There was a problem hiding this comment.
🙈 is geometry the same as area?
There was a problem hiding this comment.
I didn't want to change the api contract, and it looked like setting planningAreaId is used to set a custom planning area, whereas planningAreaId is a generic calculated field in the entity, and planningAreaGeometryId is a actual column in the entity representing the assigned custom planning area
Substitute this line for a meaningful title for your changes
Overview
Please write a description. If the PR is hard to understand, provide a quick explanation of the code.
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)