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.

Sign up for GitHub

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

Jump to bottom

feat: implement logging architecture with aggregation support #101

Merged
merged 59 commits into from
Jan 22, 2025
Merged

feat: implement logging architecture with aggregation support #101

merged 59 commits into from
Jan 22, 2025

Conversation

tpmpb
Copy link
Contributor

@tpmpb tpmpb commented Jan 3, 2025

Midaz Console Pull Request Checklist

Pull Request Type

  • UI
  • Core (Business Logic)
  • Security
  • internationalization
  • Pipeline
  • Documentation

Checklist

Please check each item after it's completed.

  • I have tested these changes locally.
  • I have updated the documentation accordingly.
  • I have added necessary comments to the code, especially in complex areas.
  • I have ensured that my changes adhere to the project's coding standards.
  • I have checked for any potential security issues.
  • I have ensured that all tests pass.
  • I have updated the version appropriately (if applicable).
  • I have confirmed this code is ready for review.

Additional Notes

Obs: Please, always remember to target your PR to develop branch instead of main.

tpmpb and others added 30 commits November 16, 2024 04:07
@tpmpb
CHORE: added pinojs package
376f6bc
@tpmpb
CHORE: added nextjs pino config
cfb8def
@tpmpb
FEATURE: added logs inicial structure
1f3f1a6
@tpmpb
feature loggers improvement
d01d7a3
@tpmpb
feature loggers improvement
c16ea05
@tpmpb
feature loggers improvement
fffa551
@tpmpb
FEATURE: added logs
bd3d6ce
@tpmpb
merge
1cedb5c
@tpmpb
middleware and midaz-id implementation
5b27d40
@tpmpb
feature loggers improvement
3ec3c46
@tpmpb
implement log on portfolio
ff00791
@tpmpb
fix: pr changes
c4343e9
@tpmpb
fix: pr changes
8171ac1
@tpmpb
Merge branch develop into feature/logger
e03a7af
@tpmpb
remove unused type
fef14e3
@tpmpb
remove unused code
58eb532
@tpmpb
fix: remove console and unused codes
529b28e
@tpmpb
wip logger
8970253
@tpmpb
wip inversify logger
55e59d1
@caioaletroca
✨ feat: Updated LogOperation decorator
0e2ec55
@augusto-draxx
Merge pull request #92 from LerianStudio/develop
a0ac1ff 
merge: develop to main
@tpmpb
wip logger
f67415e
@tpmpb
wip logger
00c04ea
@tpmpb
wip added apply middleware
29e2479
@tpmpb
wip added logs params
45e05a0
@tpmpb
wip added logs params
f533320
@tpmpb
wip added logs params
6e1b7b6
@tpmpb
implement midaz id
684ef96
@tpmpb
midazid logger
05f3ba5
@tpmpb
feature: added log deletation
f33fc9b
@tpmpb tpmpb requested a review from caioaletroca January 11, 2025 03:21
@tpmpb tpmpb changed the title 🔧 feat(logs): implement logging architecture with aggregation support feature: implement logging architecture with aggregation support Jan 11, 2025
augusto-draxx
augusto-draxx requested changes Jan 13, 2025
View reviewed changes
Copy link
Contributor

@augusto-draxx augusto-draxx left a comment

Choose a reason for hiding this comment

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

Adjust feature to aggragate log.
Remove logLevel attribute from loggerMiddleware.

@tpmpb tpmpb changed the title feature: implement logging architecture with aggregation support feat: implement logging architecture with aggregation support Jan 16, 2025
@tpmpb tpmpb requested a review from augusto-draxx January 16, 2025 07:47
@tpmpb tpmpb requested a review from gabrielcastr0 January 21, 2025 14:11
@tpmpb tpmpb merged commit 34d7701 into develop Jan 22, 2025
4 checks passed
@tpmpb tpmpb deleted the feature/midaz-logger-v1 branch January 22, 2025 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@caioaletroca caioaletroca caioaletroca approved these changes

@augusto-draxx augusto-draxx augusto-draxx approved these changes

@gabrielcastr0 gabrielcastr0 gabrielcastr0 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tpmpb @caioaletroca @augusto-draxx @gabrielcastr0