Open-Assistant
Open-Assistant copied to clipboard
LAION-AI
Unifying the task structure
This moves the existing tasks to a simpler url hierarchy: /evaluate and /create. We rename the pages to match the underlying task names in the backend.
Further, this adds some UI layout to the tasks so users can navigate to the home page without struggle and fixes the UI highlighting for the rating task.
would it be simpler to use layouts for the header and footer?
https://nextjs.org/docs/basic-features/layouts
I like that idea. If this doesn't get reviewed tomorrow I'll add that in. otherwise I'll add it in another PR. If you want, could you create some layouts in a separate PR and then I'll use them?
I hope we don't end up with a lot of merge conflicts...
I am currently working on #30 and #31 (without all of the header / footer stuff), if this branch gets merged first, I can gladly add the layout in my pr.
could you please assign #30 and #31 to me? maybe also #28 for tracking purposes.
I hope we don't end up with a lot of merge conflicts...
I am currently working on #30 and #31 (without all of the header / footer stuff), if this branch gets merged first, I can gladly add the layout in my pr.
could you please assign #30 and #31 to me? maybe also #28 for tracking purposes.
It'll be a good time to test rebase vs merge practices...
Can you make a comment in #30 and #31? I can't seem to assign them to you right now but I will as soon as Github lets me.