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

Alica Freund #3

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

Alica Freund #3

wants to merge 13 commits into from

Conversation

alicak
Copy link

@alicak alicak commented Oct 29, 2024

Alica Freund

  • RE#001 completed
  • RE#002 completed
  • RE#003 completed

Task Report

The commit log from my first commit until commit 5764d96 is an accurate representation of my time spent (there was ~30 minutes of work before the first commit).

I worked on the tasks RE#001, RE#002 and RE#003 in that order.

I spent most of my time on RE#003 and especially on implementing the requirement for staff login, because that required introducing new UI components and communication among them, and most of this was new for me since I hadn't worked with React before.

I wasn't satisfied with my implementation of staff login from RE#003, so the next day I returned to it and improved it a bit, and meanwhile discovered and fixed the bug that was part of RE#002. It took ~15 minutes.

Later I spent another ~5 minutes doing the cleanup.

I left some TODOs in the code in places where I would want to add test but in the end didn't, since I already used more time than what was recommended.

Notes

Prior to starting the work, I spent ~1.5 hour reading the tasks from the backlog, browsing the repository and skimming the libraries used on the frontend. The goal was to determine what was already implemented, what was needed and what were the dependencies between the tasks, so that I could pick those which I expected to complete.

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