-
Notifications
You must be signed in to change notification settings - Fork 524
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
Create tracking_chain-milestone_2.md #989
Conversation
@FedeC87p thank you for the milestone delivery. The application worked well but it is possible to improve some documentation and I had some doubts during the evaluation. Please see the evaluation document and provide proper answers and fixes. Let me know when I can continue this evaluation. |
Hi, thanks for review, here are the answers below
I provided the Docker only for Sql server as it is the most invasive, while the other services are provided with the selfcontained packages of the dotnet framework. But if you need to test them I can also see to release a docker with a ready-to-use solution
Sorry, Account ID and ProfileGroup ID were not visible from the Admin tools, I already released a new version to fix this. Now they are visible in the Details section. I also updated the page https://github.com/TrackingChains/TrackingChain/wiki/Milestone-2-Guide-to-Demo by adding the "FrontEnd Configuration" section, where there are two configuration examples (the most common ones and which in practice they cover the whole configuration). In that guide he shows how to use AccountId and how to configure new smartccontracts to be used. I remain at your disposal for further details and clarifications. |
Hi, I just published a guide with instructions to be able to launch all services via docker https://github.com/TrackingChains/TrackingChain/wiki/Docker-Compose The images are already configured with the test data and the configuration shown in the Step by Step guide. With this I should have settled all the outstanding points and the review can continue @dsm-w3f . Thank you EDIT: I've added more details on the setup stage with a new section https://github.com/TrackingChains/TrackingChain/wiki#configuring-your-system-for-smart-contract-integration |
@FedeC87p thank you for the improvements and fixes. I updated the evaluation document and now the milestone is accepted. I'll forward your invoice internally and the payment should take place within two weeks. Great job! |
Hi @FedeC87p, I noticed that you used already in the past the same invoice number. Many thanks, |
Thanks for reporting @fededubbi , I just sent with the correct number |
hi @FedeC87p we just transferred the payment |
Milestone Delivery Checklist
Link to the application pull request: w3f/Grants-Program#1915