Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/arpa 53#3 #44

Closed
wants to merge 9 commits into from
Closed

Feature/arpa 53#3 #44

wants to merge 9 commits into from

Conversation

MichaelKaiser1
Copy link
Contributor

Translation (en,de) not working. Help requested

language: string;
captchaSuccess = false;
captchaKey = environment.captcha.key;
script: any;

constructor(formBuilder: FormBuilder,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das Captcha sollte glaub ich ins Registrierungsformular und nicht ins Login :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hatte mit Michael besprochen dass ich es hier rein mach. Geht aber natürlich auch auf der anderen Seite

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

brauchen wir das im Login @MichaelSpielmann ? Hätte es jetzt auch eher auf der Registerseite verortet um Spam-Registrierungen zu vermeiden.. beim login is ja wurscht denk ich, oder?

src/app/app.module.ts Outdated Show resolved Hide resolved
Transalation and Validation works but browser console throws exception:
Unhandled Promise rejection: reCAPTCHA has already been rendered in this element ; Zone: <root> ; Task: Promise.then ; Value: Error: reCAPTCHA has already been rendered in this element
@MichaelKaiser1
Copy link
Contributor Author

Habe jetzt noch eine andere Variante implementiert. Übersetzung funktioniert. Allerdings wird in der Konsole im Browser ein Error geworfen
test läuft leider nicht durch :(
https://github.com/Enngage/ngx-captcha#readme
https://www.npmjs.com/package/ngx-captcha

@MichaelKaiser1
Copy link
Contributor Author

Könnte sein dass das Problem von der Google API kommt
DethAriel/ng-recaptcha#123

@MichaelKaiser1 MichaelKaiser1 deleted the feature/ARPA-53#3 branch March 13, 2021 11:04
@MichaelKaiser1 MichaelKaiser1 restored the feature/ARPA-53#3 branch March 13, 2021 11:19
@MichaelKaiser1 MichaelKaiser1 deleted the feature/ARPA-53#3 branch March 13, 2021 13:51
VILLAN3LL3 pushed a commit that referenced this pull request Feb 3, 2022
…ies-to-list_2nd_try

added create/modify/rating properties
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants