-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add support for woodlot licensees #659
Comments
Checked with Sean and a woodlot license is a combination of numbers and letters, always 5 characters total (W followed by 4 numbers). |
Hi @basilv , I will crate a new code table "project_plan_code" to hold two code ("WOODLOT", "FSP") values. |
Hi @OlgaLiber2 , |
Hi @OlgaLiber2 , This is my local implementation screenshot: The result table has:
I guess probably the above questions might need some discussion with Julius/Shan. These are draft pr links in dev environment: |
One simple option is to change the column to Plan ID and add FSP # or WL # as prefix in the column values. The search by FSP ID could be left as-is, as long as its logic is correct to ignore woodlot licenses. |
That's all good questions @ianliuwk1019 Those are all design so I will let @Kghiya look into it and work with you. |
@OlgaLiber2 I have 2 options for you to select from |
I will modify previous "FSP ID" column based on the new Mock up link but label it as "Plan Number" instead of "ID" (to be similar like FOM Number) and use "WL #" as prefix rather than "WLN #". |
Current FOM Production existing fsp_id;
Wonder if there is number(s) in PROD that's for existing Woodlot FOM(s) or if not, then we are good. Should we ask Julius/Shan (I don't see 000 because the existing datatype is an Integer)? |
I suspect the 115364 FSP ID is likely an incorrect entry, by the way. And the query should return a value of "0", so if it doesn't I suspect that's because there are no woodlot licenses having created any FOM. |
@ianliuwk1019 @basilv |
* Add new migration script for project_plan_code table and woodlot_plan_number column * Add projecrt-plan entity * Add new columns 'woodlotPlanNumber' and 'projectPlanCode' to project entity. * Add projectPlan and woodlotPlanNumber properties to request/response dto. * Add ProjectPlanCode to project module * Update swagger and gen new api client. * Rename woodlotLicenseNumber column * Add woodlotLicenseNumber and fsp fom field and validations. * Rest fields when plan selection change. * Remove console log * Update projectPlan to projectPlanCode * Fix column mapping * Add readonly fields * Add WOODLOT to summary * Add woodlot license to view detail * Add woodlotLicenseNumber and projectPlanCode fields to publicSummary * Use projectPlanCodeEnum in page so no need to hardcode. * Fix some background-color for search result table * Remove unused projectPlanDescription * Fix test * Open FSP and Woodlot# for edit. * Add getProjectPlanNumber() * Remove commented out coede. * Fix indentation. * Fix script mistake. * Update swagger after merging from main * Add woodlot/fsp_id condition rendering for public notice. * align number to center
TEST deployment is done. Please have a look: Thanks @OlgaLiber2 ! |
@ianliuwk1019 do you have a FOM number with Woodlot license published in test to look at? |
Hi @OlgaLiber2 , I added one yesterday in TEST environment. (FOM:100385) |
@ianliuwk1019 It looks good. Just one thing. I think we can remove the "WL #" prefix from the table (in the UI). So it shows W4321 under Plan number. |
Hi @OlgaLiber2 , |
@ianliuwk1019 Ah, ok. I didn't realize FSP was prefixed with FSP #. So that makes sense to have WL #.... |
Describe the task
Add support for woodlot licensees to enter their woodlot number instead of FSP ID
Acceptance Criteria
Additional context
The text was updated successfully, but these errors were encountered: