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.

Sign up for GitHub

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

Jump to bottom

ACQ-2228: removing paymentTerm.updateOptions method because is not longer used #743

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

ACQ-2228: removing paymentTerm.updateOptions method because is not longer used #743

wants to merge 1 commit into from

Conversation

maguero
Copy link
Contributor

@maguero maguero commented Jul 12, 2023

Description

Removing paymentTerm.updateOptions method because is not longer used.

This change is related (but not blocked) to:

Ticket

https://financialtimes.atlassian.net/browse/ACQ-2228

Reminder

Have you completed these common tasks (remove those that don't apply)?

  • Documentation updated or created
  • Tests written for new or updated for existing functionality
  • Stories updated to use this change
  • Accessibility checked for screen readers and contrast
  • Design Review ran past the designer
  • Product Review ran past the product owner

@maguero maguero requested a review from a team as a code owner July 12, 2023 10:14
@JuanTincho JuanTincho requested a review from a team July 12, 2023 12:06
@ludrob ludrob requested a review from a team July 12, 2023 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ludrob ludrob ludrob approved these changes

@JuanTincho JuanTincho JuanTincho approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@maguero @ludrob @JuanTincho