Skip to content
This repository has been archived by the owner on Sep 26, 2022. It is now read-only.

force update the items when they are changed #206

Merged
merged 2 commits into from
Apr 6, 2021

Conversation

romanown
Copy link
Contributor

@romanown romanown commented Apr 6, 2021

Closes #188, it is enough to set a simple check condition that in fact does not change anything, but forces the component to update the data.

it is enough to set a simple check condition that in fact does not change anything, but forces the component to update the data.
@vercel
Copy link

vercel bot commented Apr 6, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/thecomputerm/svelte-materialify/DXwfjJJp3RHjQk1E6o5EUmsWv84f
✅ Preview: https://svelte-materialify-git-fork-romanown-patch-4-thecomputerm.vercel.app

@romanown romanown changed the title force update the items when its fetched force update the items when they are fetched Apr 6, 2021
@Florian-Schoenherr
Copy link
Collaborator

Now I get it, it should react when items changes..

@Florian-Schoenherr
Copy link
Collaborator

Florian-Schoenherr commented Apr 6, 2021

items && format(value) seems to work as well, could you use that? (Sorry this took so long, after that I'm gonna merge it finally)

@Florian-Schoenherr Florian-Schoenherr changed the title force update the items when they are fetched force update the items when they are changed Apr 6, 2021
@romanown
Copy link
Contributor Author

romanown commented Apr 6, 2021

ok. i do it

changed expression for simplification
@romanown
Copy link
Contributor Author

romanown commented Apr 6, 2021

I made the change you suggested

@Florian-Schoenherr Florian-Schoenherr merged commit b3d7d64 into TheComputerM:master Apr 6, 2021
@romanown
Copy link
Contributor Author

romanown commented Apr 6, 2021

many thanks to you for this. now I can safely continue to use this library in my work. I will try to be useful to this library.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(Select) empty itsems after fetching they
2 participants