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

feat: created message log component #228

Closed
wants to merge 6 commits into from
Closed

Conversation

ethantrantalis
Copy link
Collaborator

…ute for logs. Used existing log schema? Not sure if correct. Still fixing a problem where it does not link properly to user

Developer: Ethan Trantalis

Closes #207

Pull Request Summary

Created a message log component for populating the audit log page in the admin panel. this uses the existing log schema to populate.

Modifications

Dynamically populates the audit log with messages from the log tests collection of the logSchema.

Testing Considerations

Still needs linking to the user who generated the log.

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

Screenshot 2025-02-17 at 12 36 12 AM

…ute for logs. Used existing log schema? Not sure if corrent. Still fixing a problem where it does not link properly to user
@ethantrantalis ethantrantalis linked an issue Feb 17, 2025 that may be closed by this pull request
Copy link

vercel bot commented Feb 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
slo-beaver-brigade ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 24, 2025 3:44am
slo-beaver-brigade-dummy ❌ Failed (Inspect) Feb 24, 2025 3:44am

@ethantrantalis ethantrantalis changed the title feat: created log component and added pulling from db with new API ro… feat: created message log component Feb 17, 2025
Copy link
Collaborator

@SilveerDusk SilveerDusk left a comment

Choose a reason for hiding this comment

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

Please check slack dm

…seState in MessageLog.tsx and page.tsx for Audit Log to prevent a hydration error. Error no longer occurs locally for me.
…slo-beaver-brigade into message-log

Merge necessary after fixing current branch, codebase was behind remote
and needed to be remerged before pushing back to remote.
@ethantrantalis
Copy link
Collaborator Author

Creating new pr with new changes.

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