-
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
Pay Admin: fix new filing type code NWITH fees #24959
Comments
@Mihai-QuickSilverDev , can you please do this? |
@eason-pan-bc says the code is actually NWITH. If it is already provisioned then THIS ticket is not needed. Ref: #24952 |
This is the PR worked on the fee code bcgov/lear#3010 |
If @ArwenQin demonstates a working fee code in the new filing then we can close this ticket :) |
@Mihai-QuickSilverDev |
@severinbeauvais @ArwenQin |
@ArwenQin Yes, work with this for now. I will add all the company types shortly. |
@severinbeauvais |
Let's keep this ticket to track the needed additional company types (and fee update). I'm afk so I can't publish your shared repo changes until tomorrow. For now, can you work with hard coded values? |
@severinbeauvais Added for all company types in Dev. Please note that Priority Fee applies to this fee code. |
@severinbeauvais tested with BC Limited in dev , the fees is coming up as $10 |
@OlgaPotiagalova Please close this ticket if you are OK with it. |
I set the Estimate to 1 to account for time researching and testing this. |
Doesn't seem to work for BEN in DEV, when calling the fee endpoint with BEN. @severinbeauvais @Mihai-QuickSilverDev |
Pay Admin Dev is down. I need to ask @seeker25 to restore it. |
moving back to in-progress as the issue is not resolved and needs alignment on the fees |
Pay admin has a different URL I've emailed Mihai it |
The text was updated successfully, but these errors were encountered: