Skip to content
This repository was archived by the owner on Jun 7, 2025. It is now read-only.

Conversation

@bazz333
Copy link
Contributor

@bazz333 bazz333 commented May 11, 2025

No description provided.

@lucaribon lucaribon self-requested a review May 11, 2025 14:05
lucaribon
lucaribon previously approved these changes May 11, 2025
Whyle
Whyle previously requested changes May 11, 2025
Copy link
Contributor

@Whyle Whyle left a comment

Choose a reason for hiding this comment

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

da sistemare e aggiornare test's

@lucaribon lucaribon self-requested a review May 11, 2025 14:51
lucaribon
lucaribon previously approved these changes May 11, 2025
@lucaribon lucaribon self-requested a review May 11, 2025 14:59
@Whyle
Copy link
Contributor

Whyle commented May 11, 2025

per passare l'ultimo failed del test o

fai daverro che invia un email per test(che non è molto realistico, perché i dati email di registrazione sono mockati)

o

fai il mock del send_email() con MagickMock() per ritornare sempre successful

@bazz333 bazz333 dismissed Whyle’s stale review May 11, 2025 15:29

non serve

@bazz333 bazz333 merged commit 0aca32b into main May 11, 2025
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants