Skip to content

Feature: Backup highlights manually.#63

Open
mattveraldi wants to merge 2 commits intojeromepl:mainfrom
mattveraldi:main
Open

Feature: Backup highlights manually.#63
mattveraldi wants to merge 2 commits intojeromepl:mainfrom
mattveraldi:main

Conversation

@mattveraldi
Copy link

Use Case

I currently use three computers:

  1. Personal
  2. Company-provided
  3. Client-provided

I rely on highlighter extension on all of them.

The Problem

I am unable to use my personal Google account across all devices. Both the company and client computers are protected by NDAs and managed by their respective organizations, so syncing Chrome data with my private account isn't allowed.

Solution

I need a way to manually backup my highlights.

Implementation

I created an Options Page with 3 commands:

  • Download backup: saves a JSON file with the local storage contents.
  • Upload backup: the user can upload a JSON file that will be merged with local storage contents.
  • Clear data: the user can clear highlights.

This is something that I needed and will use myself, if you think it is also useful for others, feel free to review/merge this PR.

P.S. This extension is very good and I really like its UI/UX, thank you for sharing it :)

- add download backup: json file with a dump of chrome.storage.local
- add uplaod backup: upload json file that will replace the contents of chrome.storage.local
- add clear data: clear chrome.storage.local

feat: add analytics for backup actions

feat: add input validation and merge strategy
@mattveraldi mattveraldi changed the title Feature request: Backup highlights. Feature: Backup highlights manually. Oct 27, 2024
@jeromepl
Copy link
Owner

Hi @mattveraldi , thanks for the contribution!
This would complete #25, a feature that I think a lot of people would enjoy.

I'll take a look at this in more details as soon as I can :)

@mattveraldi
Copy link
Author

Hi @jeromepl , I was wondering If you had the chance to have a look at my PR

@mattveraldi
Copy link
Author

Hi, I am reaching out to know if you plan on merging it anytime soon.

@jeromepl
Copy link
Owner

Hi @mattveraldi , sorry I haven't had time to look into this yet. My first priority was to get some basic tests going and now that that's mostly done next on my list is to look at this. I will keep you updated 🙂

@Shibin-Ez
Copy link

Hi @jeromepl , why this PR is not merged yet? This could be useful for a lot of people.

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