-
Notifications
You must be signed in to change notification settings - Fork 0
Added ticket release date, ticket available and action link to event #51
Conversation
}, | ||
ticketsAvailiable: { | ||
type: GraphQLBoolean, | ||
description: 'A boolean show whether tickets are availiable', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'A boolean to show...'
type: GraphQLBoolean, | ||
description: 'A boolean to show whether tickets are availiable', | ||
}, | ||
eventActionLink: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i feel like there's potentially some duplication going on with the externalLinks
and internalLinks
. One option I think would be to extend RelatedLinkType
to be featured as a "main" event action link.
@@ -73,6 +74,14 @@ export const EventType = new GraphQLObjectType({ | |||
type: DateTimeType, | |||
resolve: mapDateTime('endDateTime'), | |||
}, | |||
ticketReleaseDate: { | |||
type: DateTimeType, | |||
resolve: mapDateTime('ticketReleaseDate'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs description
description: 'The date when the event tickets are released.', | ||
resolve: mapDateTime('ticketReleaseDate'), | ||
}, | ||
ticketsAvailiable: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo
Bonjour!
Add a ticketReleaseDate and a boolean: ticketAvailability, for the specified event. This gives all the required information to move the react.london client to different event states automatically.