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

feat(dedicated): remove paymentmean deprecated call #11642

Merged
merged 2 commits into from May 17, 2024

Conversation

MaximeBajeux
Copy link
Contributor

@MaximeBajeux MaximeBajeux commented May 6, 2024

Question Answer
Branch? master
Bug fix? no
New feature? no
Breaking change? no
Tickets MANAGER-12850
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • [] Breaking change is mentioned in relevant commits (n/a)

Description

remove paymentmean deprecated call

Related

Recreation of: #10758

ref: MANAGER-12850

Signed-off-by: benjaminlepine <benjamin.lepine@hotmail.fr>
this.getValidPaymentMethodIds = function getValidPaymentMethodIds() {
return $http
.get(`${constants.swsProxyRootPath}me/payment/method`)
.then((response) => response.data);
Copy link
Contributor

Choose a reason for hiding this comment

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

You can directly do:

Suggested change
.then((response) => response.data);
.then(({ data }) => data);

change request

ref: feat/MANAGER-12850

Signed-off-by: Maxime Bajeux <maxime.bajeux.ext@corp.ovh.com>
Copy link

sonarcloud bot commented May 7, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@darsene darsene changed the base branch from master to release/components-w21 May 17, 2024 07:27
@darsene darsene merged commit 0983d74 into release/components-w21 May 17, 2024
15 checks passed
@darsene darsene deleted the feat/MANAGER-12850 branch May 17, 2024 07:59
@darsene darsene mentioned this pull request May 17, 2024
18 tasks
sidlynx pushed a commit that referenced this pull request May 21, 2024
ref: MANAGER-12850

Signed-off-by: Maxime Bajeux <maxime.bajeux.ext@corp.ovh.com>
sidlynx pushed a commit that referenced this pull request May 21, 2024
ref: MANAGER-12850

Signed-off-by: Maxime Bajeux <maxime.bajeux.ext@corp.ovh.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants