Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Create new race form #164

Merged
merged 4 commits into from
Aug 3, 2016
Merged

Create new race form #164

merged 4 commits into from
Aug 3, 2016

Conversation

junemoretz
Copy link
Contributor

Partially complete #152 - add race manager route at /admin/races, with form for creating new races.

@qwo qwo self-assigned this Aug 1, 2016
@qwo
Copy link
Member

qwo commented Aug 1, 2016

going to review this after work.

@qwo
Copy link
Member

qwo commented Aug 2, 2016

Looking good to me, we have some relations for Geography and Candidate. Not too sure the exact one but we can group them i think by the candidate_type for the different races and populate the data from the API.

We can go over putting some flux action/dispatchers on wednesday.

@ghost
Copy link

ghost commented Aug 3, 2016

Looks good!

Extra credit: unit tests?

On 1 August 2016 21:53:04 GMT-04:00, Stanley Zheng [email protected] wrote:

Looking good to me, we have some relations for Geography and
Candidate. Not too sure the exact one but we can group them i think
by the candidate_type for the different races and populate the data
from the API.

We can go over putting some flux action/dispatchers on wednesday.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
#164 (comment)

Ryan Y.
GnuPG Key: 0x358CD775
VoIP (ZRTP-encrypted): [email protected]
IM (OTR encryption): [email protected]

@wbprice wbprice merged commit f0ada59 into Code4HR:develop Aug 3, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants