-
Notifications
You must be signed in to change notification settings - Fork 24
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
New tmux options @1password-enabled-url-filter and @1password-items-jq-filter to disable and customize filtering #13
Conversation
@yardnsm friendly ping 😃 |
Tried here. It is working smoothly :) |
First of all - thanks! While this solution solves #1, it adds some complexity to the user and thus considered more "advanced" IMO. I would suggest adding a tmux option to use the non-filtering method (the same filter that you used in this PR), for example:
I'm all in favor for also keeping the
And the README should be updated as well:
WDYT? And sorry for the delay 😅 |
Sounds good to me, I will have a look into a solution meeting your suggestions some time in the coming week. |
@camspiers any updates on this? |
…q-filter to disable and customize filtering
ef70795
to
9c32f17
Compare
@yardnsm @odelucca New version ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but I suggested something :)
Also, I was taking a look where the script gets the items from op
, why don't we develop a cache layer? In my opinion, if we disable the sudolikeaboss
integration it became pretty slow, going to the web to fetch data every time.
I suggest we open a new PR to do this. If you want I can code based on your branch a suggestion to it :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks awesome! Sorry for the annoyance, but I have one last question: WDYT about removing the sudolikeaboss
compatibility entirely? I mean, we have now the custom JQ filter option, so we can simply add a section to the README for users wanting to maintain the compatibility.
We can simply add the now legacy JQ filter that uses the URL filtering as an example of the @1password-items-jq-filter
option in the README, and remove the @1password-enabled-url-filter
option entirely.
What do you think? Sorry for the inconsistency of my comments 😅
@camspiers @odelucca
@yardnsm I agree with you. |
Sorry, too much thrash for me. |
New tmux options @1password-enabled-url-filter and @1password-items-jq-filter to disable and customize filtering #13 Originally opened by @camspiers
* Apply changes from #13 New tmux options @1password-enabled-url-filter and @1password-items-jq-filter to disable and customize filtering #13 Originally opened by @camspiers * Remove the @1password-enabled-url-filter option * Update README.md example * Update README.md
Resolves #1 by providing two new tmux options:
@1password-enabled-url-filter to disable to default filtering
@1password-items-jq-filter to fully customize filtering
Enabled URL Filter
By default, the plugin maintains compatibility with sudolikeaboss by
filtering urls using the string "sudolikeaboss://local", by setting the following, the list of items will no longer be
filtered.
Default:
'on'
Customize URL Filtering
If complete customization of url filtering is required, a
jq
filter can be provided to filter and mapitems.
Filtering by tags
Filtering by custom url
Default:
''
Items come in the following format from which the filter operates: