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

System crash when try to print a declaration which was previously incomplete and assigned to RA #6492

Closed
SyedaAfrida opened this issue Feb 7, 2024 · 7 comments · Fixed by #6549
Assignees
Labels
Milestone

Comments

@SyedaAfrida
Copy link
Collaborator

SyedaAfrida commented Feb 7, 2024

Bug description:
System crash when try to print a declaration which was previously incomplete and assigned to him

Steps to reproduce:

  1. Login to RA
  2. . Download> assign any incomplete declaration
  3. Logout
  4. Login to registrar
  5. Unassign RA from the record
  6. Complete the declaration and register
  7. Assign the record from ready to print
  8. Login to RA
  9. Go to ready to print
  10. Try to print that record

Actual result:
System crash occurs when RA try to print

Expected result:
RA should become automatically unassigned from the record as they were unassigned and the status is changed by the registrar

Note: This bug is only not appearing for Registered status or incomplete declaration. This can appear for any status or complete declaration also. But in that case there wont be system crash. For this reason the best approach would be whenever the status is changed for any declaration by registrar, RA should be automatically un assign from that record.

Screen recording:

system.crash.when.try.to.print.a.declaration.which.was.previously.incomplete.and.assigned.to.him.mp4

for complete declaration
https://www.loom.com/share/6df8dff3b9cb4cab96a323fcaf206162?sid=565cad86-17bb-4a42-8f4b-295a13e33f32

Environment:
https://login.farajaland-staging.opencrvs.org/?lang=en
https://login.farajaland-qa.opencrvs.org/?lang=en

Version:
V1.3.3, V1.4.0,

@SyedaAfrida SyedaAfrida added the Bug label Feb 7, 2024
@eduffus
Copy link
Collaborator

eduffus commented Feb 7, 2024

@SyedaAfrida Can you confirm that this only happens if the record was submitted part-complete?
What about if the record was submitted complete then follow the same flow. Does that work fine?
A bit of extra testing around the "assign" functionality would be good here e.g. cases where RA users were last assigned at logout and the status of the record gets moved on by a Registrar, then the RA logs back in and accesses the record...
@SyedaAfrida let me know if this makes sense.

@SyedaAfrida
Copy link
Collaborator Author

@eduffus I have gone through the assigned functionalities. Though there wont be system crash for complete declaration, but there are other bugs appearing like missing data in certificate and record audit. For this reason the best approach would be whenever the status is changed for any declaration by registrar, RA should be automatically unassign from that record and a toast for unassigned by registrar should appear when logged in. I have mentioned this in the ticket to avoid confusion

@SyedaAfrida SyedaAfrida changed the title System crash when try to print a declaration which was previously incomplete and assigned to him System crash when try to print a declaration which was previously incomplete and assigned to RA Feb 12, 2024
@euanmillar euanmillar added this to the 1.3.3 milestone Feb 12, 2024
@naftis naftis assigned naftis and unassigned naftis Feb 12, 2024
@naftis
Copy link
Collaborator

naftis commented Feb 13, 2024

There are 2 things to consider here, given that you have downloaded the record as f.katongo but after that k.mweene has assigned the record to himself:

  • the DownloadButton in the work queue which shows the avatar shows that you've downloaded the record instead of the correct assignment. Should it show the actual assignment when available?
  • if you have downloaded the record, you can open it by clicking it and seeing all the downloaded information. If you have network connectivity, should there be a check for the up-to-date assignment before seeing the information you've downloaded in the past?

Toasting the user as Syeda suggested can be done: when updating work queues, check that if you have downloaded records that aren't assigned to you, and toast the user and clear the downloaded record.

I'd suggest creating the toast as a separate feature unrelated to this ticket.

@SyedaAfrida
Copy link
Collaborator Author

This bug is not properly fixed

  1. when try to assign multiple declaration at a time before older one is properly assigned than downloading is not completed and shows error
FarajalandCRS.-.Google.Chrome.-.23.February.2024.mp4
  1. when a record is reinstated by themselves then shows unassigned toast and tracking id appears twice

Image

@euanmillar
Copy link
Collaborator

@SyedaAfrida I tried to reproduce what you faced here but it works fine for me. Weird.

@euanmillar euanmillar modified the milestones: 1.3.3, 1.3.4 Feb 23, 2024
@rikukissa rikukissa linked a pull request Feb 29, 2024 that will close this issue
@SyedaAfrida
Copy link
Collaborator Author

This bug is not fixed yet
When try to assign multiple declaration at a time before older one is properly assigned than downloading is not completed and shows error

Image

@SyedaAfrida
Copy link
Collaborator Author

This bug is fixed

FarajalandCRS.-.Google.Chrome.-.8.March.2024.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants