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

🐛 Fix readonly inputs #68

Merged
merged 5 commits into from
Jan 28, 2024
Merged

🐛 Fix readonly inputs #68

merged 5 commits into from
Jan 28, 2024

Conversation

codermarcos
Copy link
Owner

@codermarcos codermarcos commented Jan 28, 2024

📝 Description

This PR should fix issue #63 which allow the user type something when input is readonly.

Thanks to @robsontenorio for report it.

In addition this PR should improve our documentation.

🧪 How to validate

Run the application locally:

  • Shouldn't allow user type something when is readonly

🏷️ Related tasks

🗃️ Changes

On branch:

fix/#63/readonly-inputs

Changes to be committed:

modified: CONTRIBUTING.md
modified: README.md
modified: cypress/e2e/v4.x.x/set-mask.cy.ts

On branch main
Changes to be committed:
	modified:   README.md
On branch fix/#63/readonly-inputs
Changes to be committed:
	modified:   cypress/e2e/v4.x.x/set-mask.cy.ts
	modified:   cypress/file/index.html
	modified:   src/set-mask.ts
	modified:   tests/set-mask.test.ts
On branch fix/#63/readonly-inputs
Changes to be committed:
	modified:   CONTRIBUTING.md
@codermarcos codermarcos added the bug A confirmed bug label Jan 28, 2024
@codermarcos codermarcos self-assigned this Jan 28, 2024
@codermarcos codermarcos changed the title Fix/#63/readonly inputs 🐛 Fix readonly inputs Jan 28, 2024
…o fix/#63/readonly-inputs

Conflicts:
	src/set-mask.ts

On branch fix/#63/readonly-inputs
All conflicts fixed but you are still merging.

Changes to be committed:
	modified:   package-lock.json
	modified:   package.json
	modified:   src/get-base-configuration.ts
	modified:   src/set-mask.ts
	modified:   src/types.ts
	modified:   tests/get-base-configuration.test.ts
@codermarcos codermarcos merged commit 030573f into main Jan 28, 2024
4 checks passed
codermarcos added a commit that referenced this pull request Aug 23, 2024
codermarcos added a commit that referenced this pull request Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A confirmed bug
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant