ispconfig3_roundcube icon indicating copy to clipboard operation
ispconfig3_roundcube copied to clipboard

pass plugin : be able to enforce variable number of lower, upper, number, symbols (patch proposed)

Open Daggett- opened this issue 11 years ago • 9 comments

Hi all,

I needed to make the pass plugin able to enforce a configurable number of lower/upper/number/symbol chars.

So I modified this plugin and did a patch but I don't know how to include nor submit it there.

I also corrected french translation of this plugin.

How can I submit my patch ?

Bye for now, Dag

Daggett- avatar Dec 09 '13 17:12 Daggett-

Here you go: https://help.github.com/articles/creating-a-pull-request

thanks for the work

horfic avatar Dec 09 '13 17:12 horfic

Thanks,

I created a pull request for that.

Bye for now, Dagg

Daggett- avatar Dec 09 '13 20:12 Daggett-

If you accept it, can my real name be in the credits instead of my login name ?

What should I do for that ?

Bye for now, Denis Chupau

Daggett- avatar Dec 09 '13 20:12 Daggett-

regarding your name, no i can't, but you can set your name in github here: https://github.com/settings/profile

i will test your code changes tomorrow.

horfic avatar Dec 09 '13 21:12 horfic

I didn't find any names or credits for all the code of thoses useful plugins : is this a decision you made for the project ?

Thanks for the testings.

There may be some functionality missing : I didn't take care of the display things linked to thoses new variable parameters aside from error messages in english and french.

I mean at least the "secure password" meter and other error messages translations may be inaccurate or missing.

Perhaps it could be interesting to put those translations on Transifex under RoundCube's project ?

Bye for now, Denis

Daggett- avatar Dec 10 '13 00:12 Daggett-

the project got currently no license and no credits, although this will change with the next upcoming release of it.

so for the time beeing it is as is.

Regarding credits for the code, except of one patch for aliases and the languages, the code comes from me.

PS: Except your new pull request.

horfic avatar Dec 10 '13 09:12 horfic

Hi,

how did your tests go ?

I think my code may miss to update things for the "strength progress bar" thing : can somone confirm this and implement the missing bit ? (I don't use that strength bar and don't feel comfortable about modifying JS).

Daggett- avatar Dec 17 '13 15:12 Daggett-

testing went well, so we will use it, but i need to update the strength progress bar first, so that both things are working in harmony.

horfic avatar Dec 17 '13 23:12 horfic