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.

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

Curator: Capture and store exhibit contact details #39

Closed
ggeisler opened this issue Jan 28, 2014 · 5 comments
Closed

Curator: Capture and store exhibit contact details #39

ggeisler opened this issue Jan 28, 2014 · 5 comments
Assignees
Labels
Milestone

Comments

@ggeisler
Copy link
Contributor

The second part of the Curation: About Pages page is a section for capturing details for the contact person(s) for the site. (These details are to be displayed in the About page sidebar; see #40).

A contact is not required. But for each contact that is entered, the Name and E-mail fields should be present; e-mail should be a valid email address.

Clicking the Add additional contact button should add an additional set of name, e-mail, title, and location fields. Each set of fields also has a "Show in sidebar" checkbox associated with it.

exhibits-wireframes-2013-12-17_pdf__page_34_of_39_

@jcoyne jcoyne added this to the Backlog milestone Feb 6, 2014
@cbeer cbeer modified the milestones: Sprint 3, Backlog Feb 13, 2014
@cbeer cbeer added the Size 3 label Feb 14, 2014
@jkeck jkeck assigned jkeck and unassigned jkeck Feb 15, 2014
@jcoyne jcoyne self-assigned this Feb 18, 2014
jcoyne added a commit that referenced this issue Feb 18, 2014
@jkeck
Copy link
Contributor

jkeck commented Feb 18, 2014

It doesn't appear that there is anyway to save the current contacts without adding an addition set of contacts fields.

screen shot 2014-02-18 at 3 04 31 pm

Also, there is no way to remove a contact. If I clear out the data I filled in in the last set of fields and click "Add additional contact" (which appears to be the only way to save a contact) then the fields remain w/ the data I cleared out.

@jkeck
Copy link
Contributor

jkeck commented Feb 18, 2014

(I should mention I realize that it is not in the mockup, but I wonder if we should formalize that before shipping)

@jcoyne
Copy link
Member

jcoyne commented Feb 18, 2014

@ggeisler I built this faithful to the wireframes. Are there any revisions you'd like to make?

@jkeck
Copy link
Contributor

jkeck commented Feb 18, 2014

Heh, sorry, I meant to comment this on the PR itself, not the ticket. Ugh. Should we take the convo over there?

@jkeck
Copy link
Contributor

jkeck commented Feb 18, 2014

(copied and amended comment on PR)

jcoyne added a commit that referenced this issue Feb 19, 2014
jcoyne added a commit that referenced this issue Feb 20, 2014
jcoyne added a commit that referenced this issue Feb 20, 2014
jcoyne added a commit that referenced this issue Feb 20, 2014
@jcoyne jcoyne closed this as completed in 8b45853 Feb 20, 2014
cbeer added a commit that referenced this issue Feb 20, 2014
Added exhibit contacts. Fixes #39, Fixes #40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants