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: Add serialize_compute and unserialize_compute #1145

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

feat: Add serialize_compute and unserialize_compute #1145

wants to merge 3 commits into from

Conversation

ArneGudermann
Copy link
Contributor

fix for #1138

@ArneGudermann ArneGudermann added the Priority: Medium This issue may be useful, and needs some attention. label Apr 18, 2024
@ArneGudermann ArneGudermann added this to the ViUR-core v3.7 milestone Apr 18, 2024
@phorward phorward added the bug(fix) Something isn't working or address a specific issue or vulnerability label Apr 29, 2024
sveneberth
sveneberth requested changes May 3, 2024
View reviewed changes
src/viur/core/bones/base.py Outdated Show resolved Hide resolved
src/viur/core/bones/base.py Outdated Show resolved Hide resolved
@ArneGudermann @sveneberth
Apply suggestions from code review
a797843 
Co-authored-by: Sven Eberth <mail@sveneberth.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sveneberth sveneberth sveneberth approved these changes

@phorward phorward Awaiting requested review from phorward

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
bug(fix) Something isn't working or address a specific issue or vulnerability Priority: Medium This issue may be useful, and needs some attention.
Projects
Documentation
Status: Todo
Milestone
ViUR-core v3.7
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ArneGudermann @sveneberth @phorward