✨ Optional extra_credits in invitations

[pcrespov]

What do these changes do?

This PR introduces an optional new field extra_credits: PositiveInt to invitations that permits a PRODUCT_OWNER user to assign a number of credits to the newly created account's wallet upon confirmation.

Extra details

APIs affected are:

• services/invitations version: 1.0.2 → 1.1.0: see openapi.json specs
  • ✨ new field extra_credits to define an invitation
• services/webserver api version: 0.32.0 → 0.33.0: see openapi.json specs
  • ✨stores invitation in confirmation.data column. Loads upon confirmation and passes it to the notification so the wallets._events can add extra credits in the wallet.
  • 🐛 sends notification also when confirmation email is disabled.

Related issue/s

• Implements part of Extension of Products: credit prices, login-per-product, PRODUCT_OWNER role and invitations #4752

How to test

• Acceptance-test:
  • PO user creates invitation with extraCredits=10
  • Use invitation link
  • Complete registration and confirm to activate account
  • Login account
  • Account has a wallet with 10 credits

DevOps

None

[codecov]

Codecov Report

Merging #4850 (22d7489) into master (a8d2126) will increase coverage by 1.4%.
The diff coverage is 89.5%.

@@            Coverage Diff            @@
##           master   #4850      +/-   ##
=========================================
+ Coverage    82.9%   84.3%    +1.4%     
=========================================
  Files         724    1040     +316     
  Lines       25732   41690   +15958     
  Branches     1063    1063              
=========================================
+ Hits        21338   35168   +13830     
- Misses       4169    6297    +2128     
  Partials      225     225              

Flag	Coverage Δ
unittests	84.3% <89.5%> (+1.4%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
...rc/models_library/api_schemas_webserver/product.py	100.0% <100.0%> (ø)
...s/models-library/src/models_library/invitations.py	91.6% <100.0%> (+0.7%)	⬆️
...ons/src/simcore_service_invitations/invitations.py	100.0% <ø> (ø)
...erver/src/simcore_service_webserver/application.py	98.1% <100.0%> (ø)
...e_service_webserver/login/handlers_registration.py	90.3% <100.0%> (ø)
...erver/src/simcore_service_webserver/login/utils.py	96.3% <100.0%> (ø)
...ervice_webserver/products/_invitations_handlers.py	100.0% <ø> (ø)
...r/src/simcore_service_webserver/wallets/_events.py	100.0% <100.0%> (ø)
...e_service_webserver/login/handlers_confirmation.py	90.9% <91.6%> (ø)
...c/simcore_service_webserver/login/_registration.py	74.1% <57.1%> (ø)

... and 358 files with indirect coverage changes

[matusdrobuliak66]

Looking good 👍

[odeimaiz]

Merci!

[bisgaard-itis]

Looks great! Thanks

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

No Coverage information
0.0% Duplication

[codeclimate]

Code Climate has analyzed commit 22d7489 and detected 0 issues on this pull request.

View more on Code Climate.