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

User should be able to get their active sessions #1034

Merged
merged 1 commit into from
Mar 6, 2025

Conversation

FancMa01
Copy link
Contributor

@FancMa01 FancMa01 commented Mar 6, 2025

Description

Please provide a summary of the changes made and the issue it resolves. Include relevant context and any dependencies required for this change.

Fixes # (issue)

Developer's Checklist

Please select at least one

  • Bug fix (non-breaking change that resolves an issue)
  • Breaking change (enhancement, fix, or feature that alters existing functionality)
  • New feature (non-breaking change that adds functionality)
  • Vulnerability fix (package updates or CodeQL adjustments to enhance code security)
  • Documentation update (addition or update of documentation or helper text)
  • Other change (please explain in the comment section)

Code Quality ( All must be selected )

  • I have commented on my code, especially in complex areas
  • I have resolved any conflicts with the target branch
  • My changes do not generate new warnings
  • I have checked my code for any misspellings
  • I have ensured my code does not duplicate existing code unnecessarily

Documentation

  • No documentation changes were required
  • I have made corresponding changes to the documentation

Security

  • I have confirmed that all security checks (e.g., CodeQL) have passed
  • No user input validation was required for this change
  • All user inputs have appropriate validation, including reasonable character limits

UX

  • This change does not involve any updates to UX elements
  • Refreshing related pages results in a functional and logical state
  • Appropriate error messages are displayed when necessary

Testing

  • Existing unit tests pass locally with my changes
  • No new unit tests were necessary for my changes
  • I have added new unit tests for my changes

Reviewer Checklist

  • I have successfully pulled the branch into my local environment, initiated the project, and verified that all the checked items above have passed

Copy link
Collaborator

@ydahal1 ydahal1 left a comment

Choose a reason for hiding this comment

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

looks good

@ydahal1 ydahal1 merged commit 2a1e3a6 into dev Mar 6, 2025
2 checks passed
@ydahal1 ydahal1 deleted the mfancher/permissions-errors branch April 1, 2025 19:28
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