search

cejug / hurraa

Opensource project to resource management
GNU General Public License v2.0
16 stars 18 forks source link

Password Confirmation field added. issue #127. #133

Closed diogodias86 closed 10 years ago

diogodias86 commented 10 years ago

Password confirmation field added to close issue #127.

helio-frota commented 10 years ago

@diogodias86 Cool ! Waiting for others to check this.

diogodias86 commented 10 years ago

Hey, @eprogramming thanks!

paulojribp commented 10 years ago

Great work @diogodias86, thanks. I will make some comments and validate your PR.

paulojribp commented 10 years ago

Try to not format files with the IDE formatting method. This make all the file change and we can't identify what specifically has changed. How many spaces are your tab? The project default is 4 spaces (not tab). You can configure this in your IDE.

diogodias86 commented 10 years ago

@paulojribp, i had to change the enconding because when i ran mvn process-resources, the maven changes the messases_pt-br.properties and the characteres were wrong.

About the formatting, i'm sorry, i' used the IDE formatting. I'll use the 4 spaces now!!

paulojribp commented 10 years ago

I tested and it works. For any reason my label doesn't appears. I see the label in message.properties, the problem can be in my environment. But, no problem your PR is approved. Thanks for your contribution.

captura de tela 2014-04-02 as 14 04 08