-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Add a new Onyx entry for LAST_VISITED_PATH
#31886
Comments
Triggered auto assignment to @joekaufmanexpensify ( |
Looks like this is an internal new feature, so going to unassign myself for now. If anything related to hiring/paying contributors comes up in the issue, feel free to reassign me, and happy to handle it! |
ACTIVE_WORKSPACE_ID
ACTIVE_WORKSPACE_ID
ACTIVE_WORKSPACE_ID
ACTIVE_WORKSPACE_ID
Hi, I'll handle this issue 🚀 |
📣 @mateuuszzzzz! 📣
|
ACTIVE_WORKSPACE_ID
LAST_VISITED_PATH
@mateuuszzzzz, @hayata-suenaga Eep! 4 days overdue now. Issues have feelings too... |
@mateuuszzzzz what is the progress for this one? I remember that the last update was that you need to coordinate with Adam about how to go about implementing this requirement as it's more tied to the navigation logic after the requirement change. How the planning going? |
@mateuuszzzzz, @hayata-suenaga Huh... This is 4 days overdue. Who can take care of this? |
@mateuuszzzzz, @hayata-suenaga Still overdue 6 days?! Let's take care of this! |
@mateuuszzzzz, @hayata-suenaga 10 days overdue. Is anyone even seeing these? Hello? |
@mateuuszzzzz, @hayata-suenaga 12 days overdue. Walking. Toward. The. Light... |
@mateuuszzzzz, @hayata-suenaga 12 days overdue now... This issue's end is nigh! |
This issue has not been updated in over 14 days. @mateuuszzzzz, @hayata-suenaga eroding to Weekly issue. |
@mateuuszzzzz I'd appreciate if you could give an update on this one once you're back at the office |
the work is on progress |
this is now kinda the main tracking issue |
Actually i think this is better issue to track it #32941 |
We need to keep track of the last visited route/URL to make it possible to navigate back to the place in the app where the user was last active.
To implement this feature, we will store the last visited path/URL as a new entry in Onyx. Please follow this section of the design doc.
You can read the further details on it in this section of the design doc.
The text was updated successfully, but these errors were encountered: