Skip to content

Feature: Structure Research Logs - #112

Merged
benloh merged 3 commits into
devfrom
dev-bl/research-log
Dec 5, 2023
Merged

Feature: Structure Research Logs#112
benloh merged 3 commits into
devfrom
dev-bl/research-log

Conversation

@benloh

@benloh benloh commented Dec 1, 2023

Copy link
Copy Markdown
Collaborator

Addresses #111

We added a new logging command that provides a more structured output for research logs.

Specifically, each record now starts with:

  • Date
  • Time
  • Network Name
  • Network Address (e.g. UADDR_01)
  • Token (aka "Group ID")
  • Action (aka "Event")
    ...and is then followed by any other items that are logged.

Per Kalani's request, we've added headings to that match. Though "DataID" and "DataDetail" might not be entirely accurate for all log items. The headings at least save you the trouble of adding the headers by hand.

The output looks like this:

Date	Time	NetName	Addr	Token	Action	DataID	DataDetail
2023/11/30	17:07:25	focus	-	-	NETCREATE APPSERVER SESSION LOG for 2023/11/30 Thursday 17:07:25
2023/11/30	17:07:25	focus	-	-	---
2023/11/30	17:07:44	focus	UADDR_01	-	joined network
2023/11/30	17:07:44	focus	UADDR_01	-	getdatabase
2023/11/30	17:07:46	focus	UADDR_01	FOCUS-B-LPJ	select node	1	Manresa
2023/11/30	17:07:48	focus	UADDR_01	FOCUS-B-LPJ	select node	2	capitola

Please give it a thorough QA pass. I believe I've caught everything, but there are always possible stray items.

@benloh
benloh requested review from jdanish and kalanicraig December 1, 2023 01:16
@benloh benloh mentioned this pull request Dec 1, 2023

@jdanish jdanish left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@benloh
benloh merged commit 21e69db into dev Dec 5, 2023
@benloh
benloh deleted the dev-bl/research-log branch December 5, 2023 05:17
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.

2 participants