Fixes https://pagure.io/pagure/issue/1234
@ryanlerch if you have a minute I would appreciate your input here :)
I tried applying this patch , I am using local as the auth , after creating a new user and loging in nothing happens .
local
It works only for fas :) and the modal looks cool !
fas
@farhaan indeed, but I'm not sure how to detect when a local user logs in for the first time :(
Personally, i think this would be better as a proper page, rather than a modal, and also visible to just users that aren't logged in.
I think the modal won't fix the issue we have of the google search results not showing what a pagure instance is actaully about:
And having a first splash page will also be good for other pagure instances to explain what their hosting is about.
@ryanlerch I see these as two different things.
@pingou sorry to interject :P but page having a starter kit for new comers sounds more appealing than modal.
starter kit
@farhaan isn't that: https://docs.pagure.org/pagure/usage/first_steps.html ?
rebased
1 new commit added
:thumbsup: from me from me
Thanks for your input and the review
Pull-Request has been merged by pingou
Fixes https://pagure.io/pagure/issue/1234