Skip to content

Conversation

@djangoliv
Copy link
Collaborator

@djangoliv djangoliv commented Dec 15, 2025

This PR cleans up the code, upgrades to JupyterHub 5, and converts to service.

The 'community.general.yaml' callback plugin has been removed.
The plugin has been superseded by the option `result_format=yaml`
@djangoliv djangoliv self-assigned this Dec 15, 2025
@djangoliv djangoliv marked this pull request as draft December 15, 2025 08:43
@djangoliv djangoliv force-pushed the upgrade branch 2 times, most recently from 6428dfe to 849ffbc Compare December 15, 2025 08:57
@djangoliv djangoliv force-pushed the upgrade branch 6 times, most recently from f8dd1be to 83dde11 Compare December 19, 2025 14:51
@djangoliv djangoliv force-pushed the upgrade branch 4 times, most recently from fcd073c to bf7af72 Compare January 5, 2026 16:39
@djangoliv djangoliv force-pushed the upgrade branch 8 times, most recently from 33eba17 to f8446d6 Compare January 8, 2026 10:10
@djangoliv djangoliv marked this pull request as ready for review January 8, 2026 10:35
@djangoliv djangoliv requested a review from trungleduc January 8, 2026 10:35
@trungleduc
Copy link
Member

Please update PR description

@djangoliv
Copy link
Collaborator Author

@trungleduc They are ok with only support sqlite.

@trungleduc trungleduc added the enhancement New feature or request label Jan 9, 2026
from traitlets import Unicode

from .permissions import Permissions, PermissionsHandler, PermissionsAPIHandler
DB_URL = "sqlite:///tljh_plasma.sqlite"
Copy link
Member

Choose a reason for hiding this comment

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

let's make it configurable via an environment variable with the default value being the current one

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@djangoliv djangoliv merged commit bc29739 into master Jan 9, 2026
4 checks passed
@djangoliv djangoliv deleted the upgrade branch January 9, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants