-
Notifications
You must be signed in to change notification settings - Fork 1
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
RAM pages polishing #138
RAM pages polishing #138
Conversation
Deploying 2nicove with Cloudflare Pages
|
cc23341
to
62ca605
Compare
let { data, debug = true }: Props = $props(); | ||
// Seeing a weird bug with svelte where this component is rerendered with data being undefined | ||
// This is a workaround to prevent the component from breaking the page | ||
let { data = [], debug = true }: Props = $props(); |
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.
I'm not seeing any issues here with it removed. This must be your local environment?
I'd rather not leave code like this here 😅
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.
This is super weird, I only see the error when going from the RAM overview page to one of the other RAM pages. You're not seeing the same thing?
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.
No I don't see anything weird when navigating between the RAM page without this code.
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.
weird.. Maybe there is something different about my environment 🤔
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.
I just ran bun clean
and I'm still seeing the errors.
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.
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.
Ah actually, the above was redundant because of this line. If you remove it, you should see the error.
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.
I removed the redundent code above 👍
Cast to a number and leave as unknown
refactor: rename extended select option type
* refactor: use index in token select value * refactor: attempt fix on generic types * fix: type error in token select props * Refactor token select dafuga (#144) * debug: making selected a value * fix: type errors * fix: type error * chore: make format * chore: remove unused method --------- Co-authored-by: Dean Sallinen <7519573+deansallinen@users.noreply.github.com> * revert: update eslint.config.js --------- Co-authored-by: Daniel Fugere <danielfugere28@gmail.com>
No description provided.