Fix placekey_format_is_valid() function for invalid placekey

[medhatgayed]

placekey_format_is_valid() function throws a ValueError when a bad placekey contains multiple @ symbols

This MR fixes that issue.

[rmtrmt]

Looks good, just one request for the test case.

[medhatgayed]

@rmtrmt It looks like the build is failing because it is expecting PLACEKEY_API_KEY environment variable. Is this something I need to do on my branch? Thanks for your help.

[rmtrmt]

@rmtrmt It looks like the build is failing because it is expecting PLACEKEY_API_KEY environment variable. Is this something I need to do on my branch? Thanks for your help.

We have PLACEKEY_API_KEY set as a secret for the repository, but pull requests from forks can't run actions with the repository's secretes. If you have an API key, I think you can add it as a secret to your fork and the actions will work. If you don't have an API key, let me know and we can work something else out.

[medhatgayed]

@rmtrmt It looks like the build is failing because it is expecting PLACEKEY_API_KEY environment variable. Is this something I need to do on my branch? Thanks for your help.

We have PLACEKEY_API_KEY set as a secret for the repository, but pull requests from forks can't run actions with the repository's secretes. If you have an API key, I think you can add it as a secret to your fork and the actions will work. If you don't have an API key, let me know and we can work something else out.

@rmtrmt I added a PLACEKEY_API_KEY to my fork repository secrets, I was not able to re-run the failed jobs because someone must have write access to the repo to re-run. Do you mind re-running the failed jobs, please?

[rmtrmt]

The jobs are still failing due to key access. I'll make sure they pass from a copy of this PR from within the repo, and if they do I'll merge this PR.