Skip to content

Commit

Permalink
use URLSearchParams instead of qs
Browse files Browse the repository at this point in the history
  • Loading branch information
simkasss committed Aug 2, 2023
1 parent 86898d8 commit c470482
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 88 deletions.
72 changes: 0 additions & 72 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@
"github-slugger": "^2.0.0",
"nanostores": "^0.8.1",
"preact": "^10.16.0",
"query-string": "^8.1.0",
"react-instantsearch-hooks-web": "^6.46.0"
},
"devDependencies": {
Expand Down
5 changes: 0 additions & 5 deletions src/features/vrf/v2/components/CostTable.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -149,11 +149,6 @@ export const getGasCalculatorUrl = ({
chain: ChainNetwork
method: Props["method"]
}) => {
console.log(
`https://vrf.chain.link/api/calculator?networkName=${mainChainName}&networkType=${
networkName === mainChainName ? chain.networkType.toLowerCase() : networkName
}&method=${method === "vrfSubscription" ? "subscription" : "directFunding"}`
)
return `https://vrf.chain.link/api/calculator?networkName=${mainChainName}&networkType=${
networkName === mainChainName ? chain.networkType.toLowerCase() : networkName
}&method=${method === "vrfSubscription" ? "subscription" : "directFunding"}`
Expand Down
19 changes: 9 additions & 10 deletions src/hooks/useQueryString.ts
Original file line number Diff line number Diff line change
@@ -1,24 +1,23 @@
import { useState, useCallback, useEffect } from "preact/hooks"
import qs from "query-string"

const setQueryStringWithoutPageReload = (qsValue) => {
if (!window) return
if (typeof window === "undefined") return

const newurl = window.location.protocol + "//" + window.location.host + window.location.pathname + qsValue

window.history.replaceState({ path: newurl }, "", newurl)
}
const setQueryStringValue = (searchParamKey, value, queryString = window.location.search) => {
if (!window) return
const setQueryStringValue = (searchParamKey, value) => {
if (typeof window === "undefined") return

const currentSearchParams = new URLSearchParams(window.location.search)
currentSearchParams.set(searchParamKey, value)

const values = qs.parse(queryString)
const newQsValue = qs.stringify({ ...values, [searchParamKey]: value })
setQueryStringWithoutPageReload(`?${newQsValue}`)
setQueryStringWithoutPageReload(`?${currentSearchParams.toString()}`)
}
const getQueryStringValue = (searchParamKey) => {
if (typeof window === "undefined") return
const values = qs.parse(window.location.search)
return values[searchParamKey]
return new URLSearchParams(window.location.search).get(searchParamKey)
}

type SearchParamValue = string | string[]
Expand All @@ -38,7 +37,7 @@ function useQueryString(

useEffect(() => {
const body = document.querySelector("body")
const observer = new MutationObserver((mutations) => {
const observer = new MutationObserver(() => {
const newQueryStringValue = getQueryStringValue(searchParamKey)
if (newQueryStringValue !== value) {
setValue(newQueryStringValue)
Expand Down

0 comments on commit c470482

Please sign in to comment.