-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Unifying the task structure #76
Conversation
… types, and fixing a ui bug in the rating task
would it be simpler to use layouts for the header and footer? |
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? |
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. |
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.
lgtm, ready for merge.
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.