search

apache / trafficcontrol

Apache Traffic Control is an Open Source implementation of a Content Delivery Network
https://trafficcontrol.apache.org/
Apache License 2.0
1.03k stars 339 forks source link

Traffic OPS ZIPCode #1761

Closed RSK14 closed 6 years ago

RSK14 commented 6 years ago

Dear All,

I add to physical location unable to add 6 digital zip codes how to fix this issue. How to slow the issue? only we can add 5digital zipcode.

Thanks, Satheesh R

dneuman64 commented 6 years ago

Thanks for the issue! Unfortunately we made a bad assumption that zip codes are 5 digits because that is what they are in the US. While we work on a fix can you add just the first 5 digits of your zip code?

RSK14 commented 6 years ago

Thanks for your response

On Fri, 12 Jan 2018 at 9:22 PM, David Neuman notifications@github.com wrote:

Thanks for the issue! Unfortunately we made a bad assumption that zip codes are 5 digits because that is what they are in the US. While we work on a fix can you add just the first 5 digits of your zip code?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/apache/incubator-trafficcontrol/issues/1761#issuecomment-357275348, or mute the thread https://github.com/notifications/unsubscribe-auth/AhuO0N338r4pLh7W1vLfisDIg9E3Ys_sks5tJ3_OgaJpZM4Rb4LN .

--

Regards,

Satheesh.R

mitchell852 commented 6 years ago

according to the database, the columns in the phys_location table really have no length limit so if it were me, I'd just remove all the maxlengths from to fix the Traffic Portal

https://github.com/apache/incubator-trafficcontrol/blob/master/traffic_portal/app/src/common/modules/form/physLocation/FormPhysLocationController.js#L32

RSK14 commented 6 years ago

Okay thanks, where i need to change this configuration.

dneuman64 commented 6 years ago

It's a simple code change. If you want, I can make the change and get a Pull Request in. If you prefer to make the change, I can show you where it needs to be done.

RSK14 commented 6 years ago

Okay could you please share me

Satheesh R

On Mon, 22 Jan 2018 at 9:02 PM, David Neuman notifications@github.com wrote:

It's a simple code change. If you want, I can make the change and get a Pull Request in. If you prefer to make the change, I can show you where it needs to be done.

— You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub https://github.com/apache/incubator-trafficcontrol/issues/1761#issuecomment-359459239, or mute the thread https://github.com/notifications/unsubscribe-auth/AhuO0JfS6idWm4O322ua7f_3OKjzg5dkks5tNKnygaJpZM4Rb4LN .

--

Regards,

Satheesh.R