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

Yhdistetty
ksteimel yhdistetty 2 committia lähteestä master kohteeseen master 4 vuotta sitten
ksteimel kommentoi 4 vuotta sitten (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] kommentoi 4 vuotta sitten (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 vuotta sitten
floreal (Migrated from github.com) jätti kommentin

Maybe a selenium test would be nice.

Maybe a selenium test would be nice.
floreal (Migrated from github.com) kommentoi 4 vuotta sitten

Just a little alignment issue

				    .input_type("password")
Just a little alignment issue ```suggestion .input_type("password") ```
elegaanz (Migrated from github.com) hyväksyi nämä muutokset 4 vuotta sitten
elegaanz (Migrated from github.com) jätti kommentin

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.

Vaihe 1:

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

Vaihe 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
Ei merkkipaalua
Ei käsittelijää
1 osallistujaa
Ilmoitukset
Määräpäivä
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

Määräpäivää ei asetettu.

Riippuvuudet

Riippuvuuksia ei asetettu.

Reference: Plume/Plume#741
Ladataan…
Sisältöä ei vielä ole.