-
Notifications
You must be signed in to change notification settings - Fork 32
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
Determine the chrome/firefox profile of the client (feature) #71
Comments
@balta2ar - any thoughts on this? I could look at this and do a PR - would that be welcome? |
Sure! But before you start writing code, as the first phase, could you do a "research" phrase where you'd look into the available APIs in different browsers and estimate where this feature may incur changes in the code? Feel free to post the relevant links and ideas here. What I mean is something like a planning stage. And then when we're on the same page, it can follow with code. |
@balta2ar Hello again. I've looked at this. It doesn't seem to be possible to get the profile like this. However, I have another proposal. The problem I am trying to solve is to make each instance of the brotab extension unique, so that I know which Profile it corresponds to. There doesn't seem to be unique id associated with each instance. Would you be ok with the introduction of an option for the brotab browser extension, through which users can add a unique string to this instance? If the string is set, then, e.g.,
How does that sound? |
How would you do that? An alternative I've just thought about is the following. It's a bit hacky, but it doesn't require changes in brotab, only in your wrapper scripts. In each profile that you have, you open a tab with a magic title, e.g. |
Hi @balta2ar, I lost track of responding, but I actually made some local changes and have used them since last year. I've added "storage" to the permissions: Manifest:
Then: background.js
Via the options page Would you be happy for me to do a PR on this? |
I guess if you have the changes locally it's no harm to make a PR :) |
I've done a PR. As noted in the PR, this is initially for discussion. Needs more polishing/testing, and a test as well. |
Let us know if there's anything else we should do - we're keen to get this integrated into the code. |
I am trying to get certain tabs to open in certain profiles. E.g. in chrome,
chrome://version/
, gives meHowever, when I query this tab, I only get this:
Is there a way to access the profile? E.g.,
or maybe
Many thanks!
The text was updated successfully, but these errors were encountered: