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

3453 - Add captcha to CSV export #3582

Merged
merged 24 commits into from
Apr 19, 2021
Merged

3453 - Add captcha to CSV export #3582

merged 24 commits into from
Apr 19, 2021

Conversation

Zasa-san
Copy link
Collaborator

@Zasa-san Zasa-san commented Apr 6, 2021

fixes #3453

PR checklist:

  • Update READ.me ?
  • Update API documentation ?

QA checklist:

  • Smoke test the functionality described in the issue
  • Test for side effects
  • UI responsiveness
  • Cross browser testing
  • Code review

@Zasa-san Zasa-san force-pushed the 3453-csv-export-captcha branch from 6c5bdc3 to 45c902c Compare April 7, 2021 20:32
@Zasa-san Zasa-san force-pushed the 3453-csv-export-captcha branch from 6625194 to 5ab88d6 Compare April 14, 2021 18:22
@Zasa-san Zasa-san marked this pull request as ready for review April 15, 2021 20:54
@Zasa-san Zasa-san requested a review from konzz April 16, 2021 14:43
@konzz konzz merged commit 78e8df0 into development Apr 19, 2021
@konzz konzz deleted the 3453-csv-export-captcha branch April 19, 2021 08:16
@simonfossom
Copy link

simonfossom commented Apr 20, 2021

@Zasa-san works on android, but on ipad it shows just download.
i dont have iphone to test.

@Zasa-san
Copy link
Collaborator Author

@Zasa-san works on android, but on ipad it shows just download.
i dont have iphone to test.

@simonfossom Thanks for the feedback! Can you please check that you are not logged in when you try on other devices? In the screenshot it looks like you are logged in, since I can see the upload cloud and the settings wheel on the top right.

@simonfossom
Copy link

sorry, my mistake. all good

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.

Add captcha to CSV export (8pts)
4 participants