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

Deprecation removals #3078

Open
wants to merge 13 commits into
base: dash-3.0
Choose a base branch
from
Open

Deprecation removals #3078

wants to merge 13 commits into from

Conversation

T4rk1n
Copy link
Contributor

@T4rk1n T4rk1n commented Nov 15, 2024

  • Remove deprecated app.run_server
  • Remove legacy dynamic component loader
  • Remove dcc.LogoutButton
  • Rename reference of long to background

@T4rk1n T4rk1n changed the base branch from dev to dash-3.0 November 15, 2024 20:16
@gvwilson gvwilson added feature something new P1 needed for current cycle labels Nov 15, 2024
@T4rk1n T4rk1n marked this pull request as ready for review November 19, 2024 14:33
@T4rk1n
Copy link
Contributor Author

T4rk1n commented Nov 19, 2024

Ready for review!

@emilykl
Copy link
Contributor

emilykl commented Nov 19, 2024

Woohoo!! 🙌

It looks like app.run_server might still be used in a handful of app templates -- @LiamConnors can you confirm? Or are these templates which are not used anymore?

https://github.com/search?q=repo%3Aplotly%2Fddk-dash-docs+run_server+language%3APython&type=code&l=Python

@emilykl
Copy link
Contributor

emilykl commented Nov 19, 2024

@T4rk1n What's the backstory regarding dcc.LogoutButton? I'm not up to date

@@ -342,7 +344,7 @@ def register_callback(
flat_state,
inputs_state_indices,
prevent_initial_call,
long=long,
Copy link
Contributor

Choose a reason for hiding this comment

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

@T4rk1n What about this argument called long= here, is there a reason it needs to stay?

@T4rk1n
Copy link
Contributor Author

T4rk1n commented Nov 19, 2024

What's the backstory regarding dcc.LogoutButton?

The button was made for DE4 but doesn't work with DE5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature something new P1 needed for current cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants