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

additional option removing partially script name #21

Closed
mcm1957 opened this issue Feb 21, 2023 · 5 comments
Closed

additional option removing partially script name #21

mcm1957 opened this issue Feb 21, 2023 · 5 comments
Labels
enhancement New feature or request fixed
Milestone

Comments

@mcm1957
Copy link
Member

mcm1957 commented Feb 21, 2023

Original: Mic-M/ioBroker.logparser#19

Could you please add a second option to remove the script name partially. For example:

script.js.Material_Design_Widgets_Examples.AdapterStatus_2: should be only AdapterStatus_2:

@mcm1957 mcm1957 added the enhancement New feature or request label Feb 21, 2023
@ciddi89
Copy link
Contributor

ciddi89 commented Feb 22, 2023

Hello @Scrounger,
I know the issue is over one year old yet. But we are trying to fix all issues now. Can you tell me please how do you mean that? If you activating Remove script.js.Script_Name: in the instance settings, it should look like your example? Or do you want a second option that only the part script.js will be removed and the script name should still be shown?

@mcm1957 mcm1957 added the question Further information is requested label Feb 22, 2023
@Scrounger
Copy link

Scrounger commented Feb 23, 2023

My wish is that last part will be visible.

Remove script.js.Script_Name: = false:

{
		"date": "Heute 09:13",
		"severity": "<span class='logWarn logSeverity'>warn</span>",
		"from": "javascript.0",
		"message": "script.js.VIS.Independent.Charts.Technik.Energie.pvForeCast: eine Warnung",
		"ts": 1677140019208
	}

Remove script.js.Script_Name: = true:

{
		"date": "Heute 09:13",
		"severity": "<span class='logWarn logSeverity'>warn</span>",
		"from": "javascript.0",
		"message": "eine Warnung",
		"ts": 1677140083042
	}

Example how i would like to have it, only pvForeCast should be visible:

{
		"date": "Heute 09:13",
		"severity": "<span class='logWarn logSeverity'>warn</span>",
		"from": "javascript.0",
		"message": "pvForeCast: eine Warnung",
		"ts": 1677140019208
	}

or e.g. an extra property script with the fullname of the script. So i have the option to create my own layout.

@ciddi89
Copy link
Contributor

ciddi89 commented Feb 23, 2023

Aaah okay now I understand, and it makes definitely sense. Okay thanks for your example. I'll look what we can do.

@ciddi89
Copy link
Contributor

ciddi89 commented Mar 3, 2023

@Scrounger
Added option to remove only script.js. Will be available in next release. If u like to test it, install GitHub version, otherwise wait for release

@ciddi89 ciddi89 added next release in Test and removed question Further information is requested labels Mar 3, 2023
@ciddi89 ciddi89 added this to the Release 2.1.x milestone Mar 3, 2023
ciddi89 added a commit that referenced this issue Mar 5, 2023
-   (ciddi89) Added: [#24](#24) Option to remove 'COMPACT' in log entries
-   (ciddi89) Added: [#21](#21) Option to remove only 'script.js' in log entries
-   (ciddi89) Fixed: [#46](#46) Midnight function to change today/yesterday
-   (ciddi89) Fixed: [#23](#23) When nothing selected in blacklist, adapter didn't work anymore
-   (ciddi89) Other: Small code and translation improvements
@ciddi89
Copy link
Contributor

ciddi89 commented Mar 5, 2023

Added in 2.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fixed
Projects
None yet
Development

No branches or pull requests

3 participants