Tab-Session-Manager icon indicating copy to clipboard operation
Tab-Session-Manager copied to clipboard

Add Group Information to Tracking

Open TheOverpassArsonist opened this issue 2 years ago • 3 comments

Adds tab group information to the tracking updates to prevent them from being wiped, fix for issue #1136

I've programmed a bit as a hobby but this is my first time doing any javascript and first time using git at all really so, while I did some testing and it seemed to work, there is a non-zero chance I've missed something here. If I have then I likely won't be able to help much in pinning the issue down.

TheOverpassArsonist avatar Nov 13 '23 06:11 TheOverpassArsonist

Maybe you can assign the repo owner (@sienori) as a reviewer for this PR so we can have a feedback on the proposed changes?

Ziip-dev avatar Nov 27 '23 15:11 Ziip-dev

Maybe you can assign the repo owner (@sienori) as a reviewer for this PR so we can have a feedback on the proposed changes?

It looks like only an owner of the repository can actually do that (according to this stack overflow answer at least) Like I said while I know a decent amount of general programming, this is my first foray into github/git for actual collaboration so I could be missing something though.

edit : noticed that stack overflow post was nearly a decade old so I looked around and found the official GH documentation on it here, it looks like while it's possible for non-owners to request reviews it's something that has to be explicitly permissioned for. (which, in fairness, is a pretty reasonable measure to prevent spam, even if it is a bit inconvenient)

TheOverpassArsonist avatar Nov 29 '23 21:11 TheOverpassArsonist

Yes, I tried creating an issue to check that and you're right: the permission is not granted by default. So we just have to hope that @sienori can have a look on your PR soon, it seems that the last interactions on PRs date back to early September.

Ziip-dev avatar Dec 01 '23 17:12 Ziip-dev

Thank you for the pull request, and I apologize for the long wait. The issue of tab groups not being saved in the tracking session was an oversight on my part. I will include this fix in the next version.

sienori avatar May 27 '24 15:05 sienori

This officially makes your extension the best browser extension ever! I'll definitely consider a donation now that I know the project is not abandoned and given how much it helps me with my "compulsive-tab-opening-disorder"! My computer's browser and I thank you for your work :)

Ziip-dev avatar May 27 '24 15:05 Ziip-dev

Hey @sienori, thank you so much for your dedication and this new release ! Cheers !

ngirard avatar May 27 '24 17:05 ngirard