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

Google API key is not needed to embed maps. Updated schema.org protocol to https. Added inline links to Schema documentation. #63

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

DaveyJake
Copy link
Contributor

@DaveyJake DaveyJake commented Jun 13, 2024

Description

  1. Google API key is not needed to embed maps.
  2. Updated Schema.org protocol to https.
  3. Added docblock inline links to Schema documentation.

Pre-review Checklist

  • Issue and pull request titles are properly formatted.
  • Acceptance criteria have been satisfied and marked in the related issue.
  • Unit tests are included (if applicable).
  • Self-review of code changes has been completed.
  • Self-review of UX changes has been completed.
  • Review has been requested from @polevaultweb.

@DaveyJake
Copy link
Contributor Author

No idea why PHPCS is saying I've got one fail even though the branch has zero conflicts.

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.

1 participant