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

Remove "OceanKnowledge Tagger" from "Submit Best Practices" drop down. #176

Closed
paulpilone opened this issue May 16, 2022 · 4 comments
Closed
Assignees

Comments

@paulpilone
Copy link
Collaborator

No description provided.

@paulineobps
Copy link
Collaborator

OK - BUT reCAPTCHA has a error.
When you click to select OCEANKNOWLEDGE TAGGER It opens but there is no obvious way to close it (except if you know to click on the OBPS logo takes you to the landing page)

@paulpilone
Copy link
Collaborator Author

@paulineobps The reCAPTCHA will not work in dev because it's configured based on a specific domain (search.oceanbestpractices.org). Since dev uses a generic CloudFront URL (d2fdejtifs20g8.cloudfront.net) the reCAPTCHA shows an error. A few thoughts here:

  • I don't think the search bar should appear on this page.
  • Figure out whether we should hide the reCAPTCHA on develop or if we can configure it with a develop and production domain name.
  • Determine a way to make navigation back to the landing page more obvious than just the OBPS logo or back button.
  • We should really verify that this page still works as expected. It's been awhile since we've tested this feature.

However, I think we should file a new issue for this and page since it's out of scope of the original request this Issue describes. I'll file that issue but I don't think we should hold up deployments because of it.

@paulpilone
Copy link
Collaborator Author

Created #227 to track the issues I mentioned in my previous comment.

@paulineobps
Copy link
Collaborator

Agree with all your above comments. I tried tagger in production and does not work there

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

No branches or pull requests

2 participants