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

병합
ksteimel master 에서 master 로 2 commits 를 머지했습니다 4 년 전
ksteimel 코멘트됨, 4 년 전 (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] 코멘트됨, 4 년 전 (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) 검토됨 4 년 전
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) 코멘트됨, 4 년 전

Just a little alignment issue

				    .input_type("password")
Just a little alignment issue ```suggestion .input_type("password") ```
elegaanz (Migrated from github.com) 이 변경사항을 승인하였습니다. 4 년 전
elegaanz (Migrated from github.com) left a comment

Thanks, I'm merging it!

Thanks, I'm merging it!

리뷰어

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
로그인하여 이 대화에 참여
No reviewers
마일스톤 없음
담당자 없음
참여자 1명
알림
마감일
기한이 올바르지 않거나 범위를 벗어났습니다. 'yyyy-mm-dd'형식을 사용해주십시오.

마감일이 설정되지 않았습니다.

의존성

No dependencies set.

Reference: Plume/Plume#741
불러오는 중...
아직 콘텐츠가 없습니다.