Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

feat: minor updates per the latest Tiles API #97

Merged
merged 1 commit into from
May 24, 2021
Merged

feat: minor updates per the latest Tiles API #97

merged 1 commit into from
May 24, 2021

Conversation

pjenvey
Copy link
Member

@pjenvey pjenvey commented May 21, 2021

Description

  • click_url's aespFlag parameter is deprecated
  • note os-family isn't currently for targeting
  • include more metadata in sentry errors

Testing

Fixes dev/stage to return tiles

Issue(s)

Closes #96

- click_url's aespFlag parameter is deprecated
- note os-family isn't currently for targeting
- include more metadata in sentry errors

Closes #96
@pjenvey pjenvey requested a review from a team May 21, 2021 22:45
.query_pairs()
.map(|p| p.0.to_string())
.collect::<Vec<String>>();
query_keys.sort();
.collect::<HashSet<String>>();
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not that sets matter much here, mostly just killing the unneeded sort() and moving it to lazy_static

@pjenvey pjenvey merged commit 44386c0 into main May 24, 2021
@pjenvey pjenvey deleted the feat/96 branch May 24, 2021 17:07
Trinaa pushed a commit that referenced this pull request Jun 13, 2022
…pes-requests-2.25.11

Bump types-requests from 2.25.10 to 2.25.11 in /client
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

adM new API integration Part 2
2 participants