Skip to content

Conversation

@lmarini
Copy link
Member

@lmarini lmarini commented Oct 15, 2024

…er adding, but

only one was on the server (refreshing showed one). Combining the useEffect statements seems to have fixed that.
At some point we should remove the complex if logic included with the jsx. We might want to have separate components for dataset and files and a common one for the actual entries.

I am actually not sure this fixes the problem. Leaving it open while I try to figure things out.

While this doesn't solve the problem (there is some kind of race condition), it does clean up the code. Marking it ready.

…er adding, but

only one was on the server (refreshing showed one). Combining the useEffect statements
seems to have fixed that.
At some point we should remove the complex if logic included with the jsx. We might want
to have separate components for dataset and files and a common one for the actual entries.
@lmarini lmarini requested a review from longshuicy as a code owner October 15, 2024 20:47
@lmarini lmarini marked this pull request as draft October 15, 2024 21:39
@lmarini lmarini marked this pull request as ready for review November 5, 2024 15:10
@lmarini lmarini merged commit 4ea5404 into release/v2.0-beta-3 Nov 5, 2024
@lmarini lmarini deleted the fix/add-metadata branch November 5, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants