@T4rk1n Sorry for the delay, I think this is ready for another look. Thanks!

@hinnefe2 I think your merge with master went bad, there is a line that should be in __init__ but is in the add_url method. Move back line 225-226 in __init__.

Thanks, looks good now. 💃

Since this is for wrapping the routes in flask_login, you could add in the add_url method a way to save the route names. You can append the url name to self.routes, it is not used presently. Just make sure it is instanciated before the _add_url calls since it is at line 215. I would use that to solve plotly/dash-auth#59 if you do that.

Then instead of subclassing Dash, you can wrap the routes externally:

for view_name, view_method in (
        (x, app.server.view_functions[x])
        for x in app.routes
        if x in app.server.view_functions
    ):
    app.server.view_functions[view_name] = login_required(view_method)

@rmarren1 can you have a look too ?

I like this change. I don't see a drawback to defining add_url as a method of Dash rather than an internal function of __init__, and should give advanced users more flexibility.

Not sure why routes doesn't do anything right now, I think we should populate that in _add_url then we can merge this
💃

@T4rk1n Ok, I think this is ready for another look. Thanks again for your patience!

@hinnefe2 Great.

We got a new CONTRIBUTING guide, make sure to you have everything on the pre-merge checklist (update version.py and changelogs) to accelerate the merge/release process.

Alright, changelog is updated and version is bumped, anything else for me to do?

Looks great, thanks! 🎊

Released dash==0.28.2