Skip to content
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

Relay graph subscription table #535

Open
soup20 opened this issue Jan 9, 2023 · 8 comments · May be fixed by #556
Open

Relay graph subscription table #535

soup20 opened this issue Jan 9, 2023 · 8 comments · May be fixed by #556
Assignees

Comments

@soup20
Copy link

soup20 commented Jan 9, 2023

Work to be done on Relay.

Context:

Building the structure for private company knowledge graphs.

Goal:

Create a table of graph subscriptions in relay that will store all the knowledge graphs a company is subscribed to (has permission to access), plus some info about each graph.

Details:

We need the following columns:

  • graph name
  • graph address (where do we send the query to?)
  • weight for graph (each graph will have it's own weight)
  • active status (is the subscription to this graph active?)
  • tribe
  • host name?
  • path for LSAT?

*Subscriptions table needs to know which LSAT to present.

@tobi-bams
Copy link
Contributor

Time estimation: 1 day but this is after we've gotten the model specification correctly with all necessary permissions needed.
@soup20

@tobi-bams
Copy link
Contributor

Thinking about this again and I am suggesting we have subscriptions linked to tribes since users would mostly visit knowledge graphs through tribes.

Based on the subscription linked to a tribe, I would know the particular graph a user is allowed to search from per tribe

@tobi-bams
Copy link
Contributor

tobi-bams commented Jan 16, 2023

@soup20, please what do you mean by host name???

@soup20
Copy link
Author

soup20 commented Jan 16, 2023

@tobi-bams That is a 'nice to have', since there is a question mark next to it. Do we have some kind of id about the host of the graph that we can use? Like maybe the company name. If we don't have anything to use here, don't worry about it.

@tobi-bams
Copy link
Contributor

@soup20 for now we don't have provisions for that, but we could always add this later

@tobi-bams
Copy link
Contributor

Status: Created a PR already (Done)

@kevkevinpal
Copy link
Contributor

@tobi-bams is this feature finished can you link the PR's for this?

@tobi-bams tobi-bams linked a pull request Feb 11, 2023 that will close this issue
@tobi-bams
Copy link
Contributor

@kevkevinpal, this is not the actual PR, the actual PR has been closed but this PR extends the other PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants