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

Include log metadata in logs table. #8419

Merged
merged 16 commits into from
Oct 9, 2024

Conversation

kenzieschmoll
Copy link
Member

@kenzieschmoll kenzieschmoll commented Oct 8, 2024

Screenshot 2024-10-08 at 4 34 28 PM

This PR displays log metadata as chips in the logs table. This includes log level (severity), which we had the data for but were not using until this PR. Fixes #1108. Fixes #6796

Work towards #7703.

);
}

extension SizeExtension on MetadataChip {
Copy link
Member Author

Choose a reason for hiding this comment

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

moved from metadata.dart

@@ -167,3 +168,46 @@ class _LoggingTableRowState extends State<LoggingTableRow> {
);
}
}

class WhenMetaDataChip extends MetadataChip {
Copy link
Member Author

Choose a reason for hiding this comment

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

moved from metadata.dart. I expect we will remove this v2 code eventually.

@kenzieschmoll kenzieschmoll marked this pull request as ready for review October 9, 2024 20:38
@kenzieschmoll kenzieschmoll requested review from bkonyi and a team as code owners October 9, 2024 20:38
@kenzieschmoll kenzieschmoll changed the title Include log details and metadata in logs table. Include log metadata in logs table. Oct 9, 2024
@kenzieschmoll kenzieschmoll merged commit 9e59cbb into flutter:master Oct 9, 2024
23 checks passed
@kenzieschmoll kenzieschmoll deleted the loggingV2 branch October 9, 2024 23:14
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.

Coloured logs would be nice Logging from developer.log properties to devtools
2 participants