Skip to content

Conversation

@xlo-u
Copy link
Contributor

@xlo-u xlo-u commented Oct 8, 2023

Summary

The password generator generates passwords of different lengths and can include or exclude special characters based on user selection. wuth the help of HTML layout, CSS styling, and JavaScript functions.

Screenshot

Screenshot 2023-10-08 124354

@xlo-u
Copy link
Contributor Author

xlo-u commented Oct 8, 2023

Ram Ram , brother.
Thank you for your previous suggestion.
I have created a new PR and hope that it meets your expectations
Thank you for taking the time to review my work

Copy link
Collaborator

@PBJI PBJI left a comment

Choose a reason for hiding this comment

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

Hi @xlo-u , your repository requires some more changes before it can be merged:


❌ DIRECTORY: Your project files, including the README.md should be inside another folder called xlo-u, so ideally your pull request changes would look something like this:

javascript-mini-projects
|           
-------- PasswordGenerator
           |
            ----- xlo-u
                   |
                    -------- **Your project files and folders**
                   |
                    --------- README.md (optional yet preferred)

Once you make these necessary changes, please revert back with a comment mentioning @PBJI so that I get notified.

Thank you. 😄

@PBJI PBJI merged commit f2ea27d into thinkswell:master Oct 8, 2023
@PBJI
Copy link
Collaborator

PBJI commented Oct 8, 2023

I have reverted the merge, it was an human error in review process. Consider the above changes and revert back with new pull request. @xlo-u

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