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

[ Feature ] - display parent note structure in note header #256

Open
2 of 3 tasks
TatianaFomina opened this issue Jul 6, 2024 · 0 comments · May be fixed by #275
Open
2 of 3 tasks

[ Feature ] - display parent note structure in note header #256

TatianaFomina opened this issue Jul 6, 2024 · 0 comments · May be fixed by #275
Assignees
Labels
good first issue this issue is good for beginners

Comments

@TatianaFomina
Copy link
Contributor

TatianaFomina commented Jul 6, 2024

This issue is related to the api issue
make sure, that api part is done

Problem

image
In disingn system we have note parents structure on note page

Cases

Structure Presentation
RootParent > Actual note RootParent > Actual note
RootParent > IntermediateNote > ActualNote RootParent > IntermediateNote > ActualNote
RootParent > IntermediateNote1 > IntermediateNote2 > ActualNote RootParent > ... > Actual note

Solution

  • Make method in useNote service that would format data got from api into presentation format
  • Update NoteDTO entity for it to contain new object with note parents info ( optional get rid of parentNote )
NoteParents : 
[
  {
    noteId: Note['id'],
    content: Note['content']
  }
]
  • Update Note page ( usage of NoteHeader component here ) as in disingn system
    note that note names should be clickable
@e11sy e11sy changed the title Display parent note in note header [ Feature ] - display parent note structure in note header Aug 8, 2024
@e11sy e11sy added the good first issue this issue is good for beginners label Aug 8, 2024
@dependentmadani dependentmadani self-assigned this Oct 26, 2024
@dependentmadani dependentmadani linked a pull request Nov 7, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue this issue is good for beginners
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants