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

[IOPLT-517] trial system API #378

Merged
merged 8 commits into from
Jun 27, 2024
Merged

[IOPLT-517] trial system API #378

merged 8 commits into from
Jun 27, 2024

Conversation

CrisTofani
Copy link
Contributor

Warning

PR depends on pagopa/io-backend#1119

Short description

PR introduces the trial system related handlers

How to test

Check both the new handlers to verify

@CrisTofani CrisTofani requested a review from a team as a code owner May 16, 2024 13:47
Copy link

codecov bot commented May 16, 2024

Codecov Report

Attention: Patch coverage is 47.61905% with 11 lines in your changes missing coverage. Please review.

Project coverage is 69.58%. Comparing base (33ca3dd) to head (4f69b6e).
Report is 6 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #378      +/-   ##
==========================================
- Coverage   69.66%   69.58%   -0.09%     
==========================================
  Files         156      159       +3     
  Lines        5008     5044      +36     
  Branches      605      612       +7     
==========================================
+ Hits         3489     3510      +21     
- Misses       1518     1533      +15     
  Partials        1        1              
Files Coverage Δ
src/server.ts 100.00% <100.00%> (ø)
src/features/trialSystem/routers/index.ts 45.00% <45.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 85d3b9d...4f69b6e. Read the comment docs.

Copy link

dpulls bot commented May 20, 2024

🎉 All dependencies have been resolved !

@CrisTofani CrisTofani merged commit f4fdc90 into master Jun 27, 2024
8 checks passed
@CrisTofani CrisTofani deleted the IOPLT-517-trial-system-api branch June 27, 2024 12:56
CrisTofani added a commit to pagopa/io-app that referenced this pull request Jun 27, 2024
…5794)

> [!warning]
> This PR depends on #5777 

## Short description
This PR aims to add a new screen with CTA to handle the loading status
of a specific trial under profile section

## List of changes proposed in this pull request
- Adds the TrialSystemPlayground under Playground section of
Profile/DeveloperSection

## How to test
Try to start the subscription by using this PR code on dev-server
pagopa/io-dev-api-server#378

---------

Co-authored-by: Mario Perrotta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants