Skip to content

2021csb1184/sof 117 save and favorite option for summary notes apis integration#149

Merged
Dhananjay-Goel merged 10 commits intoArker123:mainfrom
kritika220703:2021csb1184/sof-117-save-and-favorite-option-for-summary-notes-apis-integration
Apr 9, 2024
Merged

2021csb1184/sof 117 save and favorite option for summary notes apis integration#149
Dhananjay-Goel merged 10 commits intoArker123:mainfrom
kritika220703:2021csb1184/sof-117-save-and-favorite-option-for-summary-notes-apis-integration

Conversation

@kritika220703
Copy link
Collaborator

@kritika220703 kritika220703 commented Mar 19, 2024

This PR adds save and favorite option for summary notes

@linear
Copy link

linear bot commented Mar 19, 2024

@kritika220703
Copy link
Collaborator Author

Github Services for this month finished, workflow not working

@Arker123
Copy link
Owner

This PR appears to include commit history from #148. Let's revisit this PR after #148 is merged.

@Arker123 Arker123 removed the draft label Mar 24, 2024
@Arker123
Copy link
Owner

Arker123 commented Mar 24, 2024

Please resolve the conflicts and provide a description to the PR. Thanks

@kritika220703
Copy link
Collaborator Author

Please resolve the conflicts and provide a description to the PR. Thanks

Conflicts are resolved

Copy link
Owner

@Arker123 Arker123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please provide a description to the PR. Thanks.

kritika220703 and others added 3 commits March 31, 2024 22:53
Co-authored-by: Arnav Kharbanda <94680887+Arker123@users.noreply.github.com>
Co-authored-by: Arnav Kharbanda <94680887+Arker123@users.noreply.github.com>
Co-authored-by: Arnav Kharbanda <94680887+Arker123@users.noreply.github.com>
kritika220703 and others added 5 commits April 7, 2024 17:03
Co-authored-by: Arnav Kharbanda <94680887+Arker123@users.noreply.github.com>
Co-authored-by: Arnav Kharbanda <94680887+Arker123@users.noreply.github.com>
Copy link
Owner

@Arker123 Arker123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks

Copy link
Collaborator

@Dhananjay-Goel Dhananjay-Goel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

made the changes, lgtm
thanks

@Dhananjay-Goel Dhananjay-Goel merged commit a9e74e5 into Arker123:main Apr 9, 2024
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.

3 participants