-
Notifications
You must be signed in to change notification settings - Fork 48
48 create addedit task #74
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
Closed
dburkhart07
wants to merge
141
commits into
Code-4-Community:main
from
Code-4-Community:48-create-addedit-task
Closed
48 create addedit task #74
dburkhart07
wants to merge
141
commits into
Code-4-Community:main
from
Code-4-Community:48-create-addedit-task
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
setup & create task, category
sorry, updated this after I pushed so just gonna change it here too
Create label entity and corresponding migration
Mn 15/setup and seeding
…ndling and write tests
add npm run test
Create CODEOWNERS
implemented DueDate component
…rybuttons component
…ttons so they should be aligned now
category buttons
AA-49: Open Edit Task
38 create label popup
…d, as well as the entire form working. Still need to work on taking the data in CreateEditTasks and calling apis to update those accordingly on the save button, as well as a few UI changes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ℹ️ Issue
Closes Code-4-Community/jumpstart#48
📝 Description
For this ticket, I created the creating/adding task UI, implementing all the proper features and properly communicating with the backend. The logistics for doing this was very complicated, and the difference in creating/editing a task required a lot of manipulation with parameters between function. Regardless, I did this by creating variables for everything needing to be updated in the main file (CreateEditTask), and then passing these variables into various components that were part of the UI, and updating those in those components. I then made all the api calls that I needed in the save button function. I also added a delete button which was not on the Figma to delete a task, as we did not have that yet it was still useful.
Briefly list the changes made to the code:
✔️ Verification
I verified each of these changes by making sure each component would properly edit, and that the UI matched the Figma (aside from a few inconsistencies I took creative freedom with :D). I also performed many regression tests, ensuring that moving tasks still worked.
Task Board:


Create/Edit Task UI:
🏕️ (Optional) Future Work / Notes