-
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
Add properties related to root node and SNS node #226
Comments
@blcham Here is the example of tree that i get from server side (GET http://localhost:1235/fta-fmea/services/fta-fmea-server/faultTrees/instance1443295929). And i am not sure if i can use some properties for task except probability and upperBound Mb i don't see something ?
|
- add criticality in sns partonomy - add external reference node to fta example
… fault tree - fix in fault tree entity descriptor - explicitly read lazy relations - remove unnecessary eager relations before returning response
…ault-event-properties [FIX #226] Add missing data for fault event properties
accidentally closed. |
I added missing properties to example, see PR #243. |
…-fault-event-properties [FIX kbss-cvut/fta-fmea-ui#226] Fix reading fault event properties in fault tree
@kostobog Hi, is it correct example ? Looks like i don't have any new props. Also i don't have supertypes. Here is my server response:
|
…-fault-event-properties--2 [FIX kbss-cvut/fta-fmea-ui#226] Read fault tree in transaction to enable lazy loading
@blcham @Kasmadei I fixed two issues to make it work.
|
@kostobog I have updated BE. Used docker compose pull, cleared DB + repo sync, but unfortunately i don't see any changes :( Can you explain pls how am I supposed to update the server? |
@Kasmadei Steps:
If this does not work, try to delete the containers and volumes associated with the |
@kostobog Thank you, it works! |
@kostobog
Is there something else that i should use ? |
The following properties are not important for now:
The property |
@kostobog Hi, i wanted to ask about help with "ATA System". As u know after endpoint (/fta-fmea/services/fta-fmea-server/faultTrees/instance1443295929) call, FE also call JsonLdUtils.compactAndResolveReferences(response.data, CONTEXT); function that transforms JSON. In a CONTEXT i added: criticality: VocabularyUtils.PREFIX + "criticality", But for some reason i am not able too se "behavior" prop after transformation. Here is part of object after transformation: I can't figure out what I'm doing wrong. Don't have enough experience working with context :( |
@kostobog Can you please check this small PR: https://github.com/kbss-cvut/fta-fmea-ui/pull/251/files |
@Kasmadei, next time, I suggest not referencing uri $PR/files but rather PR directly (i.e., without "/files" suffix) . It is because your link does not connect PR to this issue (i.e. you will not see link from the PR to this issue, which is not good for a person trying to close open PRs as he might not be able to find out what is the PR about -- description of the PR is not there which is not the problem if linked to this issue) Thus, use #251 instead. |
@Kasmadei Hi,
To extend the context correctly check the context in the response, e.g. I merged #253 which adds |
Thanks, didn't know about it :) |
PR: #256 |
To show it in UI you should have data in internal-auth deployment. Moreover, consider commenting on Figma for issues related to design.
A/C:
The text was updated successfully, but these errors were encountered: