-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
User filter when searching for logs prevents Automated Process logs from showing up #10
Comments
As a developer or administrator on big orgs i would rather see my own logs by default. |
Makes sense, happy to submit a PR, any instructions on how to go about building the extension? |
from react folder
then load chrome/app as an extension in chrome |
Ok, thanks. I noticed this also affects the improved Debug Logs view (not only the separate UI you provide). I assume the code that extends the SF classic UI with the extra buttons, rows, limit, etc. is somewhere else? |
guessing here. But how does the code in |
Automated Process
logs to show up
for the old view just this file need changing and no need for build step. so you can load |
Yes it is mostly duplicated.. |
That's exactly where I was pointing earlier. My point is, it seems like that code is duplicated between the |
The code for the SF classic UI view is duplicated. |
Ok, when creating the PR shouldn't it then copy the changed code in
`chrome/app` to `firefox/app`?
…On Tue, Mar 3, 2020 at 4:25 AM Moti Ko ***@***.***> wrote:
The code for the SF classic UI view is duplicated.
*Additional* code for the tab is populated from react folder with npm
build .
If your change is SF classic UI you only need to change code in chrome/app
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#10?email_source=notifications&email_token=AACN36LAAZQSSJCYYP2PJUTRFSWF3A5CNFSM4K7W5IWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENSMIMY#issuecomment-593806387>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACN36NNH4EJ5YNTUELESR3RFSWF3ANCNFSM4K7W5IWA>
.
|
It is not a must, but if you can do same changes for firefox too would be nice. |
@robertoandrade you need any help with that? |
Even though I have
Automated Process
on the list of traced entities, since it doesn't have a hyperlink to the user it doesn't include it on the filtered logs.I wonder if the user filter is really necessary since the default implementation from SF doesn't filter the listing of logs other than by the search criteria.
The text was updated successfully, but these errors were encountered: