-
Notifications
You must be signed in to change notification settings - Fork 1
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
Incorporate feedback form into oclweb2 #653
Comments
Yes. What would be the end action of feedback? Send mail to someplace? or something else? |
|
… swagger as well
@jamlung-ri @paynejd This is deployed on QA. Try it out and let me know if there is any feedback on Feedback :) |
@snyaggarwal I'm using Firefox. It works well when I'm already signed in, and I got a confirmation email about my feedback. However, when I signed out and tried it, there were a couple of things I noticed:
|
Suggest requiring email + name if not signed in
…On Thu, Mar 18, 2021 at 8:15 AM Joseph Amlung ***@***.***> wrote:
@snyaggarwal <https://github.com/snyaggarwal> I'm using Firefox. It works
well when I'm already signed in, and I got a confirmation email about my
feedback.
However, when I signed out and tried it, there were a couple of things I
noticed:
- There was no captcha.
- It ran a bit slower than when I was signed in. (I'm working on
Firefox, but it ran faster on Edge. Not sure what to think about that.)
- It might be good to add an email address to the form so OCL can get
in contact with the feedback submitter.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#653 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJCOONAZZMW3JYXDTPJ7KTTEHVHDANCNFSM4ZI46B2Q>
.
|
…aptcha for non-logged-in users
@paynejd @jamlung-ri -- |
Thanks @snyaggarwal, it seems to be working better on Firefox today. Everything looks good, although the name and email entry fields just look a little short in terms of height. Just a small visual thing though, and everything else is good. |
@jamlung-ri Fixed |
This seems to be fixed, although there sill may be some performance issues on Firefox. @snyaggarwal and I have discussed it though, so I will close out this ticket for now. |
@snyaggarwal you mentioned that incorporating a simple feedback form into oclweb2 would be really easy -- that would be a great thing to add before launch if it's a small level of effort. Ideally, we could use a component that could also be implemented in wordpress.
The text was updated successfully, but these errors were encountered: