#1384 Add a welcome screen to new users of pagure
Merged by pingou. Opened by pingou.
welcome_screen  into  master

Download 1384.patch

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 .

It works only for fas :) and the modal looks cool !

@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:

Screenshot_from_2016-10-05_10-00-22.png

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.

  • having a front-page explaining to everyone what pagure is
  • having a welcome page/modal helping new-comers to get started with pagure (ie: they already know what pagure is, just need indication on how/where to set-up their ssh key and their emails)

@pingou sorry to interject :P but page having a starter kit for new comers sounds more appealing than modal.

@farhaan isn't that: https://docs.pagure.org/pagure/usage/first_steps.html ?

rebased

1 new commit added

  • Small grammar improvements suggested by @tibbs, thanks!

rebased

1 new commit added

  • Make the instance name configurable and reword a little the welcome screen

:thumbsup: from me from me

Thanks for your input and the review

Pull-Request has been merged by pingou

Metadata