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
Frontend queue offset #1581
base: main
Are you sure you want to change the base?
Frontend queue offset #1581
Conversation
Tested locally, looks good, working both on the detail page and in the exclusion editor. |
When a queue is growing, sometimes entering the offset resets it back to lower value even though queue is now longer. |
1791cc6
to
77ca5ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from the extra console.log, looks good!
@@ -31,7 +31,9 @@ export class Observable extends LitElement { | |||
} | |||
|
|||
disconnectedCallback(): void { | |||
console.log("disconnect?"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's leave this out, unless you want to debug it in prod?
Resolves #1580
Changes
Enables users to jump to a URL position in the crawl queue.
Manual testing
Screenshots