dekay2323 / pro3Issues

Issues for Pro3
0 stars 0 forks source link

Add another filed to Requests #135

Closed cmcisaac closed 7 years ago

cmcisaac commented 7 years ago

This would be an additional field when adding the request and an additional column on the front Material Request Table

Field is Procurement Type and the options can be:

Material Service Other

NickCrowston commented 7 years ago

I like this one but what would fall under "other" isn't everything either a material or service?

On Sat, Jul 22, 2017 at 10:30 PM, cmcisaac notifications@github.com wrote:

Assigned #135 https://github.com/dekay2323/pro3Issues/issues/135 to @NickCrowston https://github.com/nickcrowston.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dekay2323/pro3Issues/issues/135#event-1175065914, or mute the thread https://github.com/notifications/unsubscribe-auth/AbkWxQgCAi7WdAcO06c6JioxGvCIlzHdks5sQsxKgaJpZM4OgWoz .

cmcisaac commented 7 years ago

I just put other in as a default in case there is something that we arent thinking of.

Maybe someone wants to procure software and sort it separately from material and services,

I am fine with dropping other, it it probably a low probability situation.

NickCrowston commented 7 years ago

Sounds good to me

On Sun, Jul 23, 2017 at 9:41 AM cmcisaac notifications@github.com wrote:

I just put other in as a default in case there is something that we arent thinking of.

Maybe someone wants to procure software and sort it separately from material and services,

I am fine with dropping other, it it probably a low probability situation.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/dekay2323/pro3Issues/issues/135#issuecomment-317261860, or mute the thread https://github.com/notifications/unsubscribe-auth/AbkWxTC-Vacy51PuIiF13NIQhnJ92VEiks5sQ2mSgaJpZM4OgWoz .

dekay2323 commented 7 years ago

Sorry do you mean per Line Item on a request, or just on the request, so an extra column here? screenshot

cmcisaac commented 7 years ago

Just to the main request as show in your screenshot.

NickCrowston commented 7 years ago

I think it can just be called type

On Tue, Jul 25, 2017 at 8:16 PM dekay2323 notifications@github.com wrote:

What should this field be called

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/dekay2323/pro3Issues/issues/135#issuecomment-317927063, or mute the thread https://github.com/notifications/unsubscribe-auth/AbkWxfTGr5-jYHZWD8E5xP-5mr0uwWVwks5sRqFfgaJpZM4OgWoz .

dekay2323 commented 7 years ago

Done and added