Multiple field-to-finish codes#9
Conversation
|
Cursor Agent can help with this pull request. Just |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing touches🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🔴 There was an error checking the CLA! If this is your first contribution, please send in a Contributor License Agreement. |
862c948 to
9c39fc1
Compare
Description
This Pull Request introduces support for multiple "Field-to-finish" export descriptions per layer, addressing the need for greater flexibility in mapping layer features to survey/export codes. Previously, the system only allowed a single code per layer.
The changes enable the Cesium viewer to:
Entityclass now stores an array offieldToFinishCodesand aselectedFieldToFinishCode.fieldToFinishCodeasExtendedData. If multiple codes were available, allfieldToFinishCodesare also included for reference.FieldToFinishutility module provides helper functions for managing these codes and generating the necessary UI.Note: The modification to the input box for Field-to-finish export descriptions in the attribute templates (as described in the task) is an upstream change; this PR implements the consumption and display of that multi-value data within the Cesium viewer and its export functionality.
Issue number and link
No specific issue number was provided.
Testing plan
npm run lintandnpm run prettierpass.selectedFieldToFinishCode.selectedFieldToFinishCode(or the auto-resolved single code) asExtendedData.fieldToFinishCodes(comma-separated) asExtendedDatawhen multiple options exist.Author checklist
CONTRIBUTORS.mdCHANGES.mdwith a short summary of my changeSlack Thread