GitHub user Ma77Ball added a comment to the discussion: Proposal: Improve the 
dataset dashboard with a more visual card-view layout

I do like the amount of new features from the PR in the hackathon. However, 
after a quick glance, I do see that it will need significant work and review to 
be production-ready. 
1. It needs to be separated into separate PRs (too large to review in a timely 
manner).
2. Needs to leverage existing code to reduce the number of code additions 
strictly to UI/UX

> [!NOTE]
> I think my PR and the hackathon PR are guilty of this, in that too many lines 
> are added to achieve this feature, and I do agree with @carloea2 in that if 
> adding card view will add a significant amount of code to our repo, it is not 
> worth it. If either of these PRs is merged, they need to be heavily reviewed 
> and refactored. 

GitHub link: 
https://github.com/apache/texera/discussions/5130#discussioncomment-16971363

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to