-
Notifications
You must be signed in to change notification settings - Fork 59
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
Legal API: postpone NR expiry in future-effective continuation application - VERIFY #22092
Comments
@vysakh-menon-aot Can you please add a description of what needs to be updated to support FE continuation in (or other) applications/NRs? Thanks. |
It should be working for continuation in. Current implementation postpone expiry of NR for the below FE filings
|
I will verify this. |
To verify this, first I created a NR, approved it, and set the expiry date to today (Aug 29): Then I submitted a continuation application with a future effective date of tomorrow (Aug 30) at 8:30 am: Then I reviewed and approved the continuation authorization: Now I will wait until tomorrow to see if the NR expiry date was extended until past the FE date. Ref: |
Approval process has noting to do with NR expiry extension. It would have extended when user resubmited with a FE date |
Test ResultsIt appears that the NRs are being automatically expired as usual on their expiry date (but there is no record of this because expiry doesn't generate a transaction - see #23059). It also appears that the IA / continuation-in above were successfully filed/completed, even though the NR was probably expired. This is likely because of bug #22931 (which handles the NR consumption).
|
Test Results # 2
Summary: in this test, as some code was not yet fixed, the NRs show as Expired instead of Consumed. |
Test Results # 3Apparently, #23050 is now fixed, so I'm trying again...
Summary: both NRs show as Consumed, as expected! |
Test Results # 4There's a Namex error in Dev so I'm now trying in Test...
Summary: both NRs show as Consumed, as expected! |
This ticket is now verified. Still to verify: whether the NR is changed to Consumed when the applications are completed by the Filer (which will happen tomorrow at 9am). See also #23050. |
I have verified, in Dev and Test, that the NRs have been changed to Consumed status (along with a Consumed By business id). This completes the verification of this ticket. cc: @eve-git |
This code already exists but I'm not sure that it supports FE continuation in (or all other) applications.
We might also consider leveraging this code so that an application-under-review's NR doesn't expire.will be business processThe text was updated successfully, but these errors were encountered: