Routing: New useLocation()
features are still unused and (manually) implemented incorrectly in some instances
#568
Labels
The
navigate()
function (fromuseLocation()
) can be called with 3 special values:navigate("prev")
navigate("next")
navigate("up")
navigate(number)
Additionally,
useLocation()
also returns aback()
functions.There are multiple instances where these new capabilities can be used:
/notifications
or/transactions
use a?back=<path>
param that can probably be removed and replace with a simpe call toback()
, or a simplerboolean
param (e.g.?goesBack=1|0
). There's a comment about this on the back button click handler inArConnect/src/routes/popup/transaction/[id].tsx
.navigate()
function (fromuseLocation()
) includes anoption.search
param to add search params to the URL, but some usages ofnavigate()
still append it manually to the first param (make sure you tackle this issue after the previous one, as that's where most search params are being used).The text was updated successfully, but these errors were encountered: