Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Invite a user modal #3079

Closed
ErisDS opened this issue Jun 24, 2014 · 5 comments · Fixed by #3207
Closed

Invite a user modal #3079

ErisDS opened this issue Jun 24, 2014 · 5 comments · Fixed by #3207
Labels
affects:admin Anything relating to Ghost Admin
Milestone
0.5 Multi-user

Comments

@ErisDS
Copy link
Member

ErisDS commented Jun 24, 2014

Requires #3078

add button

The add button on the new users screen created in #3078 needs to be wired up to show the invite user modal:

image

To begin with author should be the only role available in the dropdown until #3083 is implemented.

The pre-styled HTML for this new screen is here: https://github.com/TryGhost/Ghost/blob/master/core/client/html/user-management.html

The original Ghost-UI issue is here: TryGhost/Ghost-UI#21
@ErisDS ErisDS added this to the 0.5 Multi-user milestone Jun 24, 2014
@ErisDS ErisDS changed the title Invite a user screen Invite a user modal Jun 24, 2014
This was referenced Jun 24, 2014
Closed
Closed
@ErisDS
Copy link
Member Author

ErisDS commented Jul 1, 2014

@morficus is grabbing this

@sebgie sebgie mentioned this issue Jul 4, 2014
Closed
@morficus
Copy link
Contributor

morficus commented Jul 5, 2014

@PaulAdamDavis is there any pre-styled HTML for this modal?
The file referenced in the description seems to only have markup for the Users Management screen.

@PaulAdamDavis
Copy link
Member

@morficus Currently, there is not. I'll get it done this morning for you. :)

You will however need to run Ghost-UI locally, using the Bower Link magic, if you aren't already.

@PaulAdamDavis
Copy link
Member

@morficus Just took #3189 for a spin, and started making this modal to style, but I it's possible we'd both do the same job, but mine wouldn't be functional.

Instead of me doing just enough to style, I think the best way to go about this, is if you make the modal, I'll pull down the PR, then style properly.

@morficus
Copy link
Contributor

morficus commented Jul 5, 2014

@PaulAdamDavis works for me. I'll probably open a PR for this later today.

@morficus morficus mentioned this issue Jul 6, 2014
Merged
morficus added a commit to morficus/Ghost that referenced this issue Jul 6, 2014
@morficus
Implementation of "invite a new user" modal
972831c 
Closes TryGhost#3079
- new controller and template for invite-new-user-modal
- actually triggers email invite via POST /ghost/api/v0.1/users/
- setting default language value (on the client) when creating a user
- only available role is "Author" - pending 3196
- updates to UsersIndexController to allow dynamic property calculation and template rending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects:admin Anything relating to Ghost Admin
Projects
None yet
Milestone
0.5 Multi-user
Development

Successfully merging a pull request may close this issue.

Implementation of "invite a new user" modal morficus/Ghost
3 participants
@ErisDS @PaulAdamDavis @morficus