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

Get exchange rates #6392

Merged
merged 5 commits into from
Jan 16, 2025
Merged

Get exchange rates #6392

merged 5 commits into from
Jan 16, 2025

Conversation

RubenGeo
Copy link
Contributor

@RubenGeo RubenGeo commented Jan 16, 2025

AB#32743

Describe your changes

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added tests wherever relevant
  • I have made sure that all automated checks pass before requesting a review
  • I do not need any deviation from our PR guidelines

Portalicious preview deployment

This PR does not have any preview deployments yet.

@RubenGeo RubenGeo added the enhancement New feature or request that affects our end users label Jan 16, 2025
@Copilot Copilot bot review requested due to automatic review settings January 16, 2025 10:58

Choose a reason for hiding this comment

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

Copilot reviewed 6 out of 11 changed files in this pull request and generated no comments.

Files not reviewed (5)
  • services/121-service/swagger.json: Language not supported
  • services/121-service/src/exchange-rates/dtos/get-exchange-rate.dto.ts: Evaluated as low risk
  • services/121-service/src/exchange-rates/exchange-rates.api.service.ts: Evaluated as low risk
  • services/121-service/src/exchange-rates/exchange-rates.controller.ts: Evaluated as low risk
  • services/121-service/src/exchange-rates/exchange-rates.module.ts: Evaluated as low risk
Comments suppressed due to low confidence (1)

services/121-service/src/exchange-rates/exchange-rates.service.spec.ts:15

  • The test descriptions should be updated to reflect the new service name 'ExchangeRatesService'.
describe('ExchangeRatesService', () => {
Copy link
Contributor

@PeterSmallenbroek PeterSmallenbroek left a comment

Choose a reason for hiding this comment

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

Looks mighty fine, small typo fix

@RubenGeo RubenGeo merged commit 22f8b7b into main Jan 16, 2025
7 checks passed
@RubenGeo RubenGeo deleted the feat.get-exchange-rates branch January 16, 2025 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request that affects our end users
Development

Successfully merging this pull request may close these issues.

2 participants