Skip to content
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

Handover Transition - SWCR status code incorrect. e.g.,, Comm. pkg. 8730-H03 #1494

Closed
EDUT-Equinor opened this issue Nov 28, 2023 · 10 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@EDUT-Equinor
Copy link

Description

Status for SWCR picking up wrong Status.

These two SWCR-status-code for Comm. pkg. 8730-H03. The old version shows "Closed - Rejected" which is correct, and the new version shows "Accepted" which is not the current status.

Reproduce

Old Handover - Correct :
image

New Handover - Incorrect:
image

Expected behavior

@EDUT-Equinor EDUT-Equinor added the bug Something isn't working label Nov 28, 2023
@Gustav-Eikaas
Copy link
Contributor

Gustav-Eikaas commented Nov 29, 2023

@BenSvend, can you verify this calculation in the SQL view?

@BenSvend
Copy link
Contributor

BenSvend commented Nov 30, 2023

@Gustav-Eikaas In the data I can see in FAM (their completion view, not one we've created). It does indeed show what is displayed on the bottom image. I don't have access to check how the calculation is made, and I therefore assume that the data is wrong. Should contact FAM for clearification.

image

@BenSvend
Copy link
Contributor

BenSvend commented Dec 1, 2023

Accodring to FAM the mistake lies with Procosys, the following is a response from Procosys:

"Status is a calculated field, based on several tables, it is not persisted. We do send the current status when sending the swcr, but perhaps that is a mistake.
This is a bug, and I'm not sure how to solve it, will have to work it out."

@ken-mellem
Copy link
Contributor

We recognize the issue - our decision is to remove the SWCR Status field from the view until we get a properly calculated field from source.

@BenSvend
Copy link
Contributor

BenSvend commented Dec 6, 2023

Closing this issue as "Wont fix". Since FAM have indicated that they will remove the Status field from their database until further notice.

Look at the issue below for more information:

@BenSvend BenSvend closed this as not planned Won't fix, can't repro, duplicate, stale Dec 6, 2023
@EDUT-Equinor
Copy link
Author

Hei @BenSvend @ken-mellem - we defo need this issue resolved somehow, have had some more comments today about it not working.

@EDUT-Equinor EDUT-Equinor reopened this Dec 13, 2023
@EDUT-Equinor
Copy link
Author

Here are some pics i got yesterday on topice - we cant have errors like these if we want users to trust new tools, that is my concern.
Old
image

New
image

Old:
image
new:
image

image
image

Same issue also happening in Query:
Old:
image
New
image
Please check related Queries in ProCoSys:
image

@ken-mellem
Copy link
Contributor

Closed as duplicate

See

@ken-mellem
Copy link
Contributor

ken-mellem commented Dec 18, 2023

@EDUT-Equinor calculation issues for Query should be reported in a new separate issue - thanks.

@EDUT-Equinor
Copy link
Author

yes whoops! i did do that but forgot to remove the query issue here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants