Fixed #734 , passowrds are now hidden on the remote_interact pages #741

Merged
ksteimel merged 2 commits from master into master 4 years ago
ksteimel commented 4 years ago (Migrated from github.com)

This is a quick fix to remove visible passwords on the remote_interact pages.

This is a quick fix to remove visible passwords on the remote_interact pages.
codecov[bot] commented 4 years ago (Migrated from github.com)

Codecov Report

Merging #741 into master will decrease coverage by 0.18%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #741      +/-   ##
==========================================
- Coverage   39.16%   38.98%   -0.19%     
==========================================
  Files          73       73              
  Lines        9653     9699      +46     
  Branches     2183     2229      +46     
==========================================
  Hits         3781     3781              
+ Misses       4819     4788      -31     
- Partials     1053     1130      +77     
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/741?src=pr&el=h1) Report > Merging [#741](https://codecov.io/gh/Plume-org/Plume/pull/741?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/02c528cae4385883d13bfff891d1656613d24edd&el=desc) will **decrease** coverage by `0.18%`. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #741 +/- ## ========================================== - Coverage 39.16% 38.98% -0.19% ========================================== Files 73 73 Lines 9653 9699 +46 Branches 2183 2229 +46 ========================================== Hits 3781 3781 + Misses 4819 4788 -31 - Partials 1053 1130 +77 ```
floreal (Migrated from github.com) reviewed 4 years ago
floreal (Migrated from github.com) left a comment

Maybe a selenium test would be nice.

Maybe a selenium test would be nice.
floreal (Migrated from github.com) commented 4 years ago

Just a little alignment issue

				    .input_type("password")
Just a little alignment issue ```suggestion .input_type("password") ```
elegaanz (Migrated from github.com) approved these changes 4 years ago
elegaanz (Migrated from github.com) left a comment

Thanks, I'm merging it!

Thanks, I'm merging it!

Reviewers

The pull request has been merged as 506fe9955d.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b master master
git pull origin master

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff master
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#741
Loading…
There is no content yet.