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

Event Data Model #11

Open
mdewolfe opened this issue Jun 4, 2013 · 11 comments
Open

Event Data Model #11

mdewolfe opened this issue Jun 4, 2013 · 11 comments

Comments

@mdewolfe
Copy link
Contributor

mdewolfe commented Jun 4, 2013

@iToto

I am going to use the example JSON array (its some where in one of the files, I forget which) of a list of events as my starting point for modelling the events.

@iToto
Copy link
Member

iToto commented Jun 4, 2013

was it in the pull request? I couldn't find it :(

@mdewolfe
Copy link
Contributor Author

mdewolfe commented Jun 4, 2013

No. I had to junk my first copy of the project. And I only noticed the sample JSON array today. Still working on it.

@mdewolfe
Copy link
Contributor Author

mdewolfe commented Jun 5, 2013

Based on the sample I found in the project, the event data model would look something like this:

PSEvent
------------
id : int
eventId : int
status : int
creationDate : NSTimeInterval (which really just a CGFloat which really just a float)
lastmodeified : NSTimeInterval
userId : int
permission : int
name : NSString
  1. I have a few questions. What is the userId intended to represent?
  2. What is the difference between the eventId and id?
  3. Are there any images that are going to be associated with any event?
  4. Is permisson needed? Why is it there?
  5. What is status? What are the possible values?

I was going to use CoreData to manage the data models, so all the ints and floats will actually be stored as NSNumbers.

I think thats enough for now.

@iToto
Copy link
Member

iToto commented Jun 6, 2013

Alright so:

  1. userId is the user that created the event. Should probably change that to creator to be more obvious
  2. not entirely sure what id is for. Looking at the ERD, there is only one id in the events table. Sooooo you can delete this lol
  3. Yes there are images that related to the event. There's an API call for that but I will be changing the end-point as @GrahamBarnard has requested here
  4. permission was added to give users the ability to set events to be read-only for certain users. Although this hasnt been implemented yet, it is in the pipeline
  5. status is also a field that I added with the intent to maybe allow users to hide images or whatnot. Not sure if this will be used, but it's there.

Hope that answers your questions. Let me know if anything is still unclear.

@mdewolfe
Copy link
Contributor Author

mdewolfe commented Jun 6, 2013

Cool. I'll keep working on it.

@mdewolfe
Copy link
Contributor Author

mdewolfe commented Jun 9, 2013

Here is preliminary version of the data model: https://github.com/mdewolfe/partyStreamApp/tree/FEATURE_models

There is folder called DataModels which contains the data models as an XML file. Have look tell me what you think. I would recommend pulling my data model branch an open the project form there. Xcode will display the entities with their relationships pretty well. Or you plug through the XML file.

Maybe create a new branch (develop?), I''l send a pull request and we can leave comments as needed before it is actually merged.

Let me know what you think.

@iToto
Copy link
Member

iToto commented Jun 11, 2013

Develop branch created :)

I'll look at the DataModels this evening.

@iToto
Copy link
Member

iToto commented Jun 12, 2013

Seems cool. Can you also add lat/long to the event and photo?

You can go ahead and pull-request it into develop :)

@mdewolfe
Copy link
Contributor Author

Will do. I will probably do it tomorrow if that's cool. I still have to generate the header files.

@iToto
Copy link
Member

iToto commented Jun 12, 2013

no rush. I still got some updates to do on the API

@mdewolfe mdewolfe mentioned this issue Jun 14, 2013
@iToto
Copy link
Member

iToto commented Jun 14, 2013

yeah this looks good to me. I'll go ahead and merge it in.

Also, I was thinking of the notion of using FourSquare. What are your thoughts? It could be used to replace the location storing, or as a compliment feature. IMO it would be nice to have both seeing as how not everyone uses FourSquare.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants