data.coop/website
8
1
Fork 6
Code Issues 11 Pull requests Releases Wiki Activity

Update description for logging badge #85

Merged
benjaoming merged 4 commits from benjaoming/website:feature/new/badge/logging into new 2023-06-04 21:02:28 +00:00
Conversation 1 Commits 4 Files changed 1 +3 -8
benjaoming commented 2023-04-15 21:17:20 +00:00
Owner
Copy link

Ref: #77

Ref: #77
benjaoming added 1 commit 2023-04-15 21:17:21 +00:00
Add description for logging
All checks were successful
continuous-integration/drone/pr Build is passing
Details
03c7d882bf
benjaoming added the
new-website
 label 2023-04-15 21:27:03 +00:00
samsapti requested changes 2023-05-20 15:54:08 +00:00
samsapti left a comment
Owner
Copy link

Ændringsforslag fra @halfd og jeg:

Bemærk at "positiv" status for logning betyder, at tjenesten laver forventet minimal logning af meta-data som vi tilstræber at beskrive for hver tjeneste. En "negativ" status betyder at der logges mere end nødvendigt.

Ændringsforslag fra @halfd og jeg: > Bemærk at "positiv" status for logning betyder, at tjenesten laver forventet minimal logning af meta-data som vi tilstræber at beskrive for hver tjeneste. En "negativ" status betyder at der logges mere end nødvendigt.
benjaoming commented 2023-06-04 20:42:28 +00:00
Author
Owner
Copy link

Jeg har justeret lidt på teksten:

  Bemærk at "positiv" status for logning betyder, at tjenesten laver forventet minimal logning af meta-data, hvilke vi tilstræber at dokumentere for hver enkelt tjeneste. En "negativ" status betyder at der logges mere end nødvendigt.

Det er stort set det samme, men jeg synes, at det blev tydeligeret "hvilke" henviser til "logning af meta-data".

Jeg forstår hensigten med rettelsen, teksten fremstår simplere. Og såklart, at der er tale om, at "minimal logning" er centralt. Jeg synes dog også, at det er vigtigt, at forståelsen af "minimal" fremlægges, så det kan diskuteres af medlemmerne :)

Jeg har justeret lidt på teksten: > Bemærk at "positiv" status for logning betyder, at tjenesten laver forventet minimal logning af meta-data, hvilke vi tilstræber at dokumentere for hver enkelt tjeneste. En "negativ" status betyder at der logges mere end nødvendigt. Det er stort set det samme, men jeg synes, at det blev tydeligeret "hvilke" henviser til "logning af meta-data". Jeg forstår hensigten med rettelsen, teksten fremstår simplere. Og såklart, at der er tale om, at "minimal logning" er centralt. Jeg synes dog også, at det er vigtigt, at forståelsen af "minimal" fremlægges, så det kan diskuteres af medlemmerne :)
benjaoming added 3 commits 2023-06-04 20:42:50 +00:00
Adds a TODO comment based on what was removed
All checks were successful
continuous-integration/drone/pr Build is passing
Details
f60bd27991
benjaoming merged commit 723e2864cb into new 2023-06-04 21:02:28 +00:00
benjaoming deleted branch feature/new/badge/logging 2023-06-04 21:02:36 +00:00
samsapti approved these changes 2023-06-10 19:09:02 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
samsapti
Labels
Clear labels   
new-website

   
ui
No labels
new-website
ui
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: data.coop/website#85
No description provided.
Delete branch "benjaoming/website:feature/new/badge/logging"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?