-
Notifications
You must be signed in to change notification settings - Fork 6
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
Implement list view for FTA trees and Systems #247
Comments
I am close to finish table. But i need to ask for some changes on backend side. To get info about tree FE calls endpoint: Current response for "/systems" is:
And for "/faultTrees" response is almost similar:
@kostobog Can I ask to add the missing properties for both endpoints? |
No, you can assume that there is "name" of the System, but that's all for now. The most important part to show now are FTA trees. |
@kostobog please implement easier part of the table and create ticket to do the rest afterwards. |
@Kasmadei I realized, that i forgot to update the image (i mentioned it on the meeting but did not update the screenshot): |
@blcham Is it correct, that there should be a dropdown with filter for all aircraft types ? So after i select some aircraft type, i'll see in table only aircrafts with selected type. |
Yes but it is not part of this ticket -- read in the screenshot "No need to do filtering yet". We do not need to do filtering nor adding fields that would realize filters there yet. |
@blcham I was thinking that you wanted to add some other changes in the table |
Please do "docker-compose pull". There are new data to be shown in the list from this PR: Here is the list of newly added stuff:
|
@Kasmadei @blcham
|
@Kasmadei make additional PR to include those fields there |
@kostobog Hi. I've received the following response from the endpoint:
|
@Kasmadei
I am no sure why there is no FHA based failure rate. I will check today and let you know. |
@kostobog Hi, any updates ? |
…and-trees-view [#247] Fixes for system table
@Kasmadei please past here a picture which fields work. |
@blcham @kostobog looks like i don't have any trees from response, can you check it please? GET I did a docker update with previously provided instruction:
|
@Kasmadei following the steps above it worked for me. Did you get some error? |
@blcham Yes, and /systems works correct, i see the same as you, but for /faultTrees endpoint response is empty :/ In an hour i'll check it again Edit: Tried from start and i see the same. Endpoint http://localhost:1235/services/fta-fmea-server/faultTrees/summaries has empty response. |
@blcham I tested running @Kasmadei Now you should be able to get all fields necessary for the table except for status:
Here is a snippet of one of the summary of one of the fault trees:
|
@Kasmadei Until this issue #276 is resolved you can deploy the example data to db-server manually. To do so first run internal-auth as usual. Visit
|
@kostobog Hi, tried your solution and for some reason i don't see fta-fmea/ava/user-example.trig file :( |
I think you forgot to do Do the following steps:
|
@blcham No, i did not forget about docker-compose build db-server, and it did not help. I'll try today again :) |
@blcham @kostobog BE updated without any issue, but response from endpoint is still empty: Response: {"@context":{},"@graph":[]} curl 'http://localhost:1235/services/fta-fmea-server/faultTrees/summaries' |
The status will be implemented in the following issue: #280 The implementation needs to be checked, and if it is okay, we can close it. |
@Kasmadei I see the following issues: |
After discussion we decided to add a new column for "created". I'll check how it will look with new column first. |
A/C:
The text was updated successfully, but these errors were encountered: