Skip to content

Conversation

@dinhlongviolin1
Copy link
Contributor

@dinhlongviolin1 dinhlongviolin1 commented Apr 2, 2025

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

  • Generalized/abstracted all related server component -> less dependent on actual server implementation
  • Fastapi integration.

Related Tickets & Documents

How to reproduce the issue

Please replace this line with instructions on how to reproduce the issue or test the feature.

Other branches or releases that this needs to be backported

Describe which projects this change will impact and that needs to be backported.

Checklist

We encourage you to keep the code coverage percentage at 80% and above.

  • This solution meets the acceptance criteria of the related issue.
  • The related issue checklist is completed.
  • This PR adds unit tests for the developed code. If not, why?
  • End-to-End tests have been added or updated. If not, why?
  • The documentation has been updated, or a dedicated issue created. (If applicable)
  • The release notes have been updated? (If applicable)

@dinhlongviolin1 dinhlongviolin1 requested review from FabienLelaquais and FredLL-Avaiga and removed request for FredLL-Avaiga April 11, 2025 14:46
@github-actions
Copy link
Contributor

This PR has been labelled as "🥶Waiting for contributor" because it has been inactive for more than 14 days. If you would like to continue working on this PR, then please add new commit or another comment, otherwise this PR will be closed in 14 days. For more information please refer to the contributing guidelines.

@github-actions github-actions bot added the 🥶Waiting for contributor Issues or PRs waiting for a long time label May 21, 2025
@github-actions github-actions bot removed the 🥶Waiting for contributor Issues or PRs waiting for a long time label May 28, 2025
Copy link
Member

@FredLL-Avaiga FredLL-Avaiga left a comment

Choose a reason for hiding this comment

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

Waow @dinhlongviolin1
Impressive


def get_flask_app(self) -> Flask:
"""Get the internal Flask application.
def get_flask_app(self):
Copy link
Member

Choose a reason for hiding this comment

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

add @deprecated ?

Copy link
Member

Choose a reason for hiding this comment

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

should we use DeprecationWarning as an exception?

# Deprecated
def _set_flask(self, flask):
raise RuntimeError(
"'_set_flask()' is deprecated. Use '_set_web_server()' instead as multiple web frameworks has been supported." # noqa: E501
Copy link
Member

Choose a reason for hiding this comment

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

are now supported

Copy link
Member

@FabienLelaquais FabienLelaquais left a comment

Choose a reason for hiding this comment

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

Excellent!

@dinhlongviolin1 dinhlongviolin1 merged commit e83e952 into develop Jun 13, 2025
165 of 166 checks passed
@dinhlongviolin1 dinhlongviolin1 deleted the feature/server-generalize branch June 13, 2025 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gui: Back-End 🖰 GUI Related to GUI ✨New feature 📝Release Notes Impacts the Release Notes or the Documentation in general

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants