Skip to content

Jenna#3

Open
stickycorner wants to merge 80 commits intoCSC207-2025F-UofT:mainfrom
PrishaPatel24:jenna
Open

Jenna#3
stickycorner wants to merge 80 commits intoCSC207-2025F-UofT:mainfrom
PrishaPatel24:jenna

Conversation

@stickycorner
Copy link
Copy Markdown

@stickycorner stickycorner commented Nov 4, 2024

No description provided.

stickycorner and others added 30 commits November 2, 2024 09:12
 Merge branch 'main' of https://github.com/PrishaPatel24/ZenTask into jenna
…I tried to add some style functionalities. It is not working yet.
iguehi-akhimien1 and others added 30 commits November 26, 2024 17:07
…rcus.

I have cleaned up the code in NotesView by refactoring.
I also adjusted the width of the program because of the notes view.
…rcus.

I have cleaned up the code in NotesView by refactoring.
I also adjusted the width of the program because of the notes view.

Comment (on merged code from main):
-Jenna, you cannot pass Note into any method in the Presenter. To update content of the note, you need to go view->controller->input boundary ->interactor->output boundary->presenter->view model->view.
- You have to update the content of the note in the in-memory DAO.
-Also, Presenter does not directly interact with View but with ViewModel.
The Presenter cannot call a method in view.
-Also, you cannot create an instance of Note in the View.
-Also, you cannot pass Note into a method in the Controller.
…shots now.

I also added a check for my use case to ensure titles of notes are unique to the note.
updated TaskState to use Task entities
iguehi.akhimien - README is done, Test class is in progress.
… confusion with Google's implemented Event entity, renamed entity.
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.

4 participants