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

Add Group Information to Tracking #1266

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheOverpassArsonist
Copy link

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.

@Ziip-dev
Copy link

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

@TheOverpassArsonist
Copy link
Author

TheOverpassArsonist commented Nov 29, 2023

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)

@Ziip-dev
Copy link

Ziip-dev commented Dec 1, 2023

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.

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.

None yet

2 participants