Skip to content

Add BookmarkService - Core Utility Service #504

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Aug 8, 2024
Merged

Add BookmarkService - Core Utility Service #504

merged 7 commits into from
Aug 8, 2024

Conversation

samad-yar-khan
Copy link
Contributor

@samad-yar-khan samad-yar-khan commented Aug 7, 2024 •
edited
Loading

Linked Issue(s)

#500

Acceptance Criteria fulfillment

  • Add BookmarkService: Add a core boomark service to fetch, set and reset all bookmarks

Proposed changes (including videos or screenshots)

  • Rename existing BookmarkType Repo Enum to CodeBookmarkType.
  • Add BookmarkTypes enum for BookmarkService.
  • Added Bookmark Service: All method to fetch and update bookmarks with common getter and setter. Add method to reset all org bookmarks.
  • Add Repo methods to bulk fetch and update bookmarks.
  • Update Bookmark.type to string type.

Further comments

@samad-yar-khan samad-yar-khan changed the title Bookmark service Add BookmarkService - Core Utility Service Aug 7, 2024
@samad-yar-khan samad-yar-khan merged commit 0c9460e into main Aug 8, 2024
3 checks passed
@samad-yar-khan samad-yar-khan deleted the bookmark-service branch August 8, 2024 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jayantbh jayantbh jayantbh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@samad-yar-khan @jayantbh