Adding last_login to FAB for Superset

Open sfirke opened this issue 2 years ago • 1 comments

I'd like to implement https://github.com/apache/superset/discussions/22185#discussioncomment-7190757, adding a last-logged-in field to Apache Superset. As far as I can tell, Superset imports UserModelView from FAB and uses it as-is. So if this modification to the default could happen in FAB, it would involve a tiny change, but if Superset broke from FAB here it would involve copying a lot of FAB code that would then be harder to maintain.

Is that correct? If so, are you open to adding last_login as a default displayed field in FAB? And is that addition as simple as I've laid out in my linked comment above?

Thanks for considering!

Oct 04 '23 19:10 sfirke

Could be a nice addition as long as it fits well on the table, feel free to open a PR.

Oct 11 '23 23:10 dpgaspar