Skip to content

T05AaronTapper#84

Closed
aarontapper wants to merge 3 commits intoFarmData2:developmentfrom
aarontapper:T05AaronTapper
Closed

T05AaronTapper#84
aarontapper wants to merge 3 commits intoFarmData2:developmentfrom
aarontapper:T05AaronTapper

Conversation

@aarontapper
Copy link
Copy Markdown

Pull Request Description
Implements features of Tutorial 5 as detailed within. This took me approximately 30 minutes


Licensing Certification

FarmData2 is a Free Cultural Work and all accepted contributions are licensed as described in the LICENSE.md file. This requires that the contributor holds the rights to do so. By submitting this pull request I certify that I satisfy the terms of the Developer Certificate of Origin for its contents.

Copy link
Copy Markdown

@braughtg braughtg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • - The "Add Item" and "Cancel" buttons show and hide the form for adding a new item.
    • - The "AddItem" and "Cancel" buttons call the doEdit method correctly.
  • - The add-item-form div is shown when editing.
  • - The v-on handlers for the text input and button call the saveItem method.
  • - The "Nice job! You've bought all your items!" message appears when the list is empty and is hidden when an item is added.
  • - The items list is initially empty.
  • - The saveItem method is added to the Vue methods.
    • - The saveItem method has been extended to clear the text input when a new item is saved.
  • - The doEdit method has been added to the Vue methods
    • - The text field is cleared if doEdit is clicked.
  • - Work is spread across multiple commits.

Nicely done!

@braughtg braughtg closed this Oct 3, 2025
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