Skip to content
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

Anbar Hussain #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

anbar-h
Copy link

@anbar-h anbar-h commented Dec 5, 2024

Anbar Hussain

  • RE#001# - completed all tasks
  • RE#002# - completed all tasks
  • RE#003# - completed all tasks

Task Report

The commit log accurately reflects the order of tasks I worked on, but the timing is not fully precise due to breaks between tasks. Initially, I spent approximately an hour setting up the project, ensuring it ran properly, and creating a launch.json file in Visual Studio Code to facilitate debugging. After that, I proceeded with completing the tasks in the following order: RE#001, RE#002, and RE#003. I spent the full 180 mins on the task (excludes project setup time).

If I had more time, I would have focused on the following improvements:

  • Testing: Implemented tests to ensure code reliability and coverage.

  • UI Enhancements: Improved the user interface validation for better user experience.

  • Performance Improvements for RE#003: Added features such as paging, sorting, and filtering to handle larger datasets efficiently. I would also have focused on optimizing the indexing to improve data retrieval speed.

Overall, I feel the tasks were completed successfully, and while some optimizations were skipped due to time constraints, the implementation is functional and aligns with the requirements. I would have done more refactoring if I had time.

Notes

React Experience: While I don’t have professional industry experience with React, I tried my best to implement the tasks with the limited knowledge I have. With more time, I would have refined my approach and further enhanced the implementation.

functionality for booking a room has been implemented with validation
check cache first, otherwise call api
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.

1 participant