Skip to content

Conversation

@inject3r
Copy link

@inject3r inject3r commented May 1, 2025

Summary

Improved the OTP validation logic by making the error message more descriptive and ensuring that the prompt.done() method is only called after receiving a valid input.

How did you test this change?

  • Tested the OTP input functionality by running the code and verifying that the error message is now more clear when no input is provided.
  • Ensured that the prompt correctly closes after a valid OTP input and asks again if the input is invalid.
  • No additional tests were required as this change only affects the user input validation logic.

@inject3r inject3r changed the title Improved OTP validation message and prompt handling [Compiler] Improved OTP validation message and prompt handling May 1, 2025
Copy link

@Hardanish-Singh Hardanish-Singh left a comment

Choose a reason for hiding this comment

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

LGTM!

@inject3r inject3r changed the title [Compiler] Improved OTP validation message and prompt handling [compiler] Improved OTP validation message and prompt handling May 2, 2025
@poteto poteto closed this May 15, 2025
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.

5 participants