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

Ubuntu 24.04 Upgrade #39

Merged
merged 3 commits into from
Oct 22, 2024
Merged

Ubuntu 24.04 Upgrade #39

merged 3 commits into from
Oct 22, 2024

Conversation

jfavellar90
Copy link
Collaborator

This PR aims to close #38

@DawoudSheraz
Copy link
Contributor

@jfavellar90 Hi, what's the status of this PR? It has been in draft for a while. Thanks

@jfavellar90
Copy link
Collaborator Author

@DawoudSheraz it is still WIP since the Tutor core PR is also WIP. Please confirm if I can jump and test Notes against the current state of the Tutor core PR.

@DawoudSheraz
Copy link
Contributor

@DawoudSheraz it is still WIP since the Tutor core PR is also WIP. Please confirm if I can jump and test Notes against the current state of the Tutor core PR.

I think you can go ahead with testing and making this PR ready for review. You can use core PR for testing if needed.

@jfavellar90 jfavellar90 marked this pull request as ready for review October 9, 2024 22:02
@jfavellar90
Copy link
Collaborator Author

@DawoudSheraz After testing with the new tutor image with Ubuntu 22.04, the Notes service worked as expected.

Copy link
Contributor

@DawoudSheraz DawoudSheraz left a comment

Choose a reason for hiding this comment

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

Please add a changelog-entry for this change.

@jfavellar90
Copy link
Collaborator Author

Please add a changelog-entry for this change.

Done @DawoudSheraz

@jfavellar90 jfavellar90 merged commit fdf73b6 into nightly Oct 22, 2024
@jfavellar90 jfavellar90 deleted the Jhony/ubuntu_24_04 branch October 22, 2024 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants