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

psp-8365 lease flexible duration #4142

Merged
merged 16 commits into from
Jul 3, 2024
Merged

Conversation

devinleighsmith
Copy link
Collaborator

No description provided.

@devinleighsmith devinleighsmith added the enhancement New feature or request label Jun 27, 2024
@devinleighsmith devinleighsmith self-assigned this Jun 27, 2024
@devinleighsmith devinleighsmith changed the title Psp 8365 psp-8365 Jun 27, 2024
@devinleighsmith devinleighsmith changed the title psp-8365 psp-8365 lease flexible duration Jun 27, 2024
@devinleighsmith
Copy link
Collaborator Author

Please note:
No new tests at this time - waiting for all major changes to payments tab first.

Copy link
Contributor

✅ No secrets were detected in the code.

1 similar comment
Copy link
Contributor

✅ No secrets were detected in the code.

Copy link
Contributor

github-actions bot commented Jul 2, 2024

✅ No secrets were detected in the code.

2 similar comments
Copy link
Contributor

github-actions bot commented Jul 3, 2024

✅ No secrets were detected in the code.

Copy link
Contributor

github-actions bot commented Jul 3, 2024

✅ No secrets were detected in the code.

@@ -28,7 +28,9 @@ export interface IPeriodsFormProps {
formikRef: React.RefObject<FormikProps<LeaseFormModel>>;
}

export const PeriodsForm: React.FunctionComponent<React.PropsWithChildren<IPeriodsFormProps>> = ({
export const PeriodPaymentsView: React.FunctionComponent<
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: please rename file to match component name: PaymentPeriodsView.tsx -> PeriodPaymentsView.tsx

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: please rename file to match component name: PaymentPeriodsView.tsx -> PeriodPaymentsView.tsx

will do in follow up PR

@devinleighsmith devinleighsmith merged commit 03278ea into bcgov:dev Jul 3, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants