[ASTPPCOM-146] Issue 142 Created: 03/Nov/16 Updated: 01/Apr/21 Resolved: 01/Apr/21 |
|
| Status: | Done |
| Project: | ASTPP Community |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | New Feature | ||
| Reporter: | countrdd | Assignee: | Samir Doshi |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
Adding option to import termination rates using a column mapper |
| Comments |
| Comment by Samir Doshi [ 10/Nov/16 ] |
|
[~countrdd] We have checked and found the issues. We have reported all the issues in, |
| Comment by countrdd [ 16/Nov/16 ] |
|
Updated. |
| Comment by dev-astpp [ 23/Nov/16 ] |
|
[~countrdd] We have checked fixed issues and give update with new found issues. We have reported all the issues in, |
| Comment by countrdd [ 23/Nov/16 ] |
|
I must not be pusing this write, because it is fine on my systems….for all the erros you said.hmmmm From: ASTPP Development notifications@github.com @countrddhttps://github.com/countrdd We have checked fixed issues and give update with new found issues. We have reported all the issues in, — |
| Comment by Samir Doshi [ 24/Nov/16 ] |
|
@countrdd, I am sure it must be working perfect in your setup with proper data. While we are getting contribution from someone, that is our duty to verify the code and features with proper and improper data and once everything fine release it under the open source version. So that community not get buggy contributions and they can adopt new changes without hassle. So far the feature which you have implemented will really be going to useful to all admins as they will no longer require to format the csvs to import the termination rates. |
| Comment by countrdd [ 25/Nov/16 ] |
|
I agree, I should have been more clear, the format changes for the screen (i.e. html) looked like what you have pointed out, and I fixed that and for some reason I am cannot seem to push that up… I am really looking forward to getting these in and figuring out what I am doing wrong with the pull request, that way I can contribute more. Dusty From: Samir Doshi <notifications@github.com> @countrdd<https://github.com/countrdd>, I am sure it must be working perfect in your setup with proper data. While we are getting contribution from someone, that is our duty to verify the code and features with proper and improper data and once everything fine release it under the open source version. So that community not get buggy contributions and they can adopt new changes without hassle. So far the feature which you have implemented will really be going to useful to all admins as they will no longer require to format the csvs to import the termination rates. — |
| Comment by countrdd [ 25/Nov/16 ] |
|
See here is where it looks ok From: Samir Doshi <notifications@github.com> @countrdd<https://github.com/countrdd>, I am sure it must be working perfect in your setup with proper data. While we are getting contribution from someone, that is our duty to verify the code and features with proper and improper data and once everything fine release it under the open source version. So that community not get buggy contributions and they can adopt new changes without hassle. So far the feature which you have implemented will really be going to useful to all admins as they will no longer require to format the csvs to import the termination rates. — |
| Comment by countrdd [ 19/Dec/16 ] |
|
Sorry, Got busy. I think everything is there, just need to know if I need to do anything else. |