dash icon indicating copy to clipboard operation
dash copied to clipboard

Remove LogoutButton

Open AnnMarieW opened this issue 1 year ago • 7 comments

Since dcc.LogoutButton is no longer recommended, it should be removed.

The docs page will be redirected to html.Button. https://github.com/plotly/ddk-dash-docs/issues/2603

This is a great "first issue" for anyone looking to contribute, and I'm happy to help you get started! If no one takes it on within about a week, I'll handle it as part of the docs PR I'm working on,

AnnMarieW avatar Jun 18 '24 16:06 AnnMarieW

Hi, I'm new open source, sorry if this is silly question, But the given link is not working.

shawon-majid avatar Jun 18 '24 17:06 shawon-majid

Hi @shawon-majid and welcome! That link is to the dash docs library, which is private. The relevant content is only about how the LogoutButton should be removed, which lead to opening this issue.

Is this something you are interested in working on?

AnnMarieW avatar Jun 18 '24 17:06 AnnMarieW

Note - This might be considered a breaking change - may need to deprecate this component, then remove it in dash 3.0

AnnMarieW avatar Jun 18 '24 17:06 AnnMarieW

Note - This might be considered a breaking change - may need to deprecate this component, then remove it in dash 3.0

Not sure we have an easy way of deprecating components, maybe we can check the class attribute (_namespace, _type) is not in a list and emit a deprecation warning in here: https://github.com/plotly/dash/blob/685dcbcb8ddb19727df87e42e655c8e33f84648f/dash/_validate.py#L412

T4rk1n avatar Jun 18 '24 17:06 T4rk1n

Hi, I've worked on a dash app on a project so I'm quite familiar with it. I'm new to open-source and would love to contribute to this issue. Could you please assign it to me?

shreya1728 avatar Jun 25 '24 15:06 shreya1728

Hi @shreya1728 there is already a PR for this issue, #2899

T4rk1n avatar Jun 25 '24 16:06 T4rk1n

@shreya1728 Thanks for your offer to contribute! Let us know if you are interested in helping with any other issues 🙂

AnnMarieW avatar Jun 25 '24 16:06 AnnMarieW

@AnnMarieW I ran into this issue very late as somebody already submitted the pr for this. I would love to contribute to other issues....could you please give me a chance?

Gannu456 avatar Nov 21 '24 09:11 Gannu456

The logout button has already been removed in dash-3.0 branch. https://github.com/plotly/dash/pull/3137#issuecomment-2631546974

andre996 avatar May 23 '25 17:05 andre996