Skip to content

fix: masked value issue in data table #3885

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.

Sign up for GitHub

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

Jump to bottom
Merged
kobenguyent merged 2 commits into from
Oct 6, 2023
Merged

fix: masked value issue in data table #3885

kobenguyent merged 2 commits into from
Oct 6, 2023

Conversation

@kobenguyent
Copy link
Collaborator

@kobenguyent kobenguyent commented Sep 22, 2023

Motivation/Description of the PR

Type of change

  • 🐛 Bug fix

Checklist:

  • Tests have been added
  • Documentation has been added (Run npm run docs)
  • Lint checking (Run npm run lint)
  • Local tests are passed (Run npm test)

@kobenguyent kobenguyent self-assigned this Sep 22, 2023
@Arhell Arhell requested a review from DavertMik September 26, 2023 00:05
@kobenguyent kobenguyent merged commit 3cc123f into 3.x Oct 6, 2023
@kobenguyent kobenguyent deleted the fix-masked-value-issue-in-data-table branch October 6, 2023 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Arhell Arhell Arhell approved these changes

@DavertMik DavertMik Awaiting requested review from DavertMik

Assignees

@kobenguyent kobenguyent

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Passwords stored with secret() are written into inputs as asterisks ("****") not as the actual string

3 participants

@kobenguyent @Arhell