search

CandyRedTele / meatballs

0 stars 0 forks source link

admin table - location as a VARCHAR #50

Closed CandyRedTele closed 10 years ago

CandyRedTele commented 10 years ago

There is a location column that is a VARCHAR ....

Values are many different cities now...

Can we change to a f_id (facility id) because that's the way we handle location. Will need to be changed in gen_admin.sql too

GeoffreyBoom commented 10 years ago

I'll do that now. Thanks.

GeoffreyBoom commented 10 years ago

Wait, didn't we say they would all be in the HQ in Montreal, since they're admins? If not, let me know.

samuelmasuy commented 10 years ago

This is true, the entire table will actually only have one location. And it shouldn't be linked to the actual restaurant situated in Montreal.

GeoffreyBoom commented 10 years ago

I'll leave it as is for now, then. I'll add facility addresses though, since Mike wanted that.

samuelmasuy commented 10 years ago

Yeah, and I need it to display it in aboutUS section.

On Wed, Aug 20, 2014 at 1:54 PM, GeoffreyBoom notifications@github.com wrote:

I'll leave it as is for now, then. I'll add facility addresses though, since Mike wanted that.

— Reply to this email directly or view it on GitHub https://github.com/jamg85/comp353-project/issues/50#issuecomment-52815618 .

CandyRedTele commented 10 years ago

Now the admins are not all working in montreal, they work in Narnia, Quebec, Winnipeg and so on...

GeoffreyBoom commented 10 years ago

I didn't change that, the default is Montreal, I will change them all to Montreal next time I update.

samuelmasuy commented 10 years ago

Well there is 2 kind of admins actually, the one working in Montreal Head office and the one working at each resto.

GeoffreyBoom commented 10 years ago

I thought we decided all admins work in Montreal? they aren't "local employees".

samuelmasuy commented 10 years ago

@jamg85 the admins working in different resto should be in localstaff table.

samuelmasuy commented 10 years ago

But yes since we decided they all work in Montreal. It's fine like this.

GeoffreyBoom commented 10 years ago

Okay, I'll put all the admins in Montreal.

GeoffreyBoom commented 10 years ago

I will also put delivery personnel in local (Y)

GeoffreyBoom commented 10 years ago

I changed all admin locations to montreal and added delivery personnel to localstaff.

CandyRedTele commented 10 years ago

Ok I think it is a close, thank yoU!