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

Samengevoegd

ksteimel heeft 2 commits samengevoegd van master naar master

ksteimel reageerde (Gemigreerd van github.com)

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

codecov[bot] reageerde (Gemigreerd van 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 (Gemigreerd van github.com) beoordeeld

floreal (Gemigreerd van github.com) heeft een reactie achtergelaten

Maybe a selenium test would be nice.

templates/remote_interact_base.rs.html

floreal (Gemigreerd van github.com) reageerde

Just a little alignment issue

				    .input_type("password")

Just a little alignment issue ```suggestion .input_type("password") ```

elegaanz (Gemigreerd van github.com) heeft deze veranderingen goedgekeurd

elegaanz (Gemigreerd van github.com) heeft een reactie achtergelaten

Thanks, I'm merging it!

kiwii verwees naar dit probleem vanuit commit '

Fixed #734 , passowrds are now hidden on the remote_interact pages (#741) * Fixed #734 , passowrds are now hidden on the remote_interact pages * Update templates/remote_interact_base.rs.html Co-Authored-By: Floréal Toumikian <floreal+github@nimukaito.net> Co-authored-by: Floréal Toumikian <floreal+github@nimukaito.net>

Reviewers

De pull request is samengevoegd als 506fe9955d.

Je kunt ook command line instructies bekijken.

Stap 1:

Vanuit het project, check een branch uit en test de veranderingen.

git checkout -b master master

git pull origin master

Stap 2:

Voeg de wijzigingen samen en update ze op Forgejo.

git checkout master

git merge --no-ff master

git push origin master

Geen beoordelaars

Geen mijlpaal

Niet toegewezen

1 deelnemers

Notificaties

Vervaldatum

De deadline is ongeldig of buiten bereik. Gebruik het formaat 'jjjj-mm-dd'.

Geen vervaldatum ingesteld.

Afhankelijkheden

Geen afhankelijkheden ingesteld.

Laden…

Er is nog geen inhoud.

Labels Mijlpalen

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

Codecov Report

Reviewers

Stap 1:

Stap 2: