Skip to content

ASSIGMENT 1 - WAN HARITH AZDY BIN WAN ISMAIL U2101138#4

Open
Wan2729 wants to merge 1 commit intomainfrom
assignment1_WanHarithAzdy
Open

ASSIGMENT 1 - WAN HARITH AZDY BIN WAN ISMAIL U2101138#4
Wan2729 wants to merge 1 commit intomainfrom
assignment1_WanHarithAzdy

Conversation

@Wan2729
Copy link
Collaborator

@Wan2729 Wan2729 commented Dec 3, 2025

Addressed Issue

  • When calling REST Endpoint for adding Tag, to the Item, the response show success even though the Item's Tag is not updated.

What I Have Reengineered

  • Added checking whether the Tag exist or not before return.
  • Added additional function to add multiple tag at once.

Reengineering Strategy or Approach Used

  • Validation-first approach: Introduced a pre-check mechanism to ensure that a tag actually exists before returning a success response.
  • Created a log message on Item status before return.
  • Added new integration test to make sure the new function work as expected.

Impact of Changes

  • Improved error handling and better logger for debugging.
  • Easier automation work by not needing to add tag one by one.

@Wan2729
Copy link
Collaborator Author

Wan2729 commented Dec 10, 2025

@suhadaudd11 Hello should we tag you or just add as reviewer is fine?

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