Skip to content
This repository has been archived by the owner on Feb 1, 2023. It is now read-only.

Add a UI for the app #7

Open
d47853 opened this issue Oct 28, 2019 · 4 comments
Open

Add a UI for the app #7

d47853 opened this issue Oct 28, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@d47853
Copy link
Member

d47853 commented Oct 28, 2019

Add a dynamic SPA which will allow users to provision and manage their Graph instances. It should use some kind of UI framework to make it easier to deliver features in the future. This should probably be either React or Angular.

@t11947
Copy link
Contributor

t11947 commented Apr 21, 2020

An MVP for the UI app should fulfil the following criteria:

  • The UI has a way of authenticating users
  • As a user, I can see a list of the graphs I can access
  • As a user, I can provision a new graph with a name and a brief description
  • As an administrator, I can list the graphs and see which users can access them
  • As an administrator, I can grant a user access to a graph
  • As an administrator, I can delete a graph

@t11947 t11947 added the enhancement New feature or request label Apr 21, 2020
@macenturalxl1
Copy link
Contributor

macenturalxl1 commented May 18, 2020

Hi - I'll claim this one please.

I'll start on the "As a user, I can see a list of the graphs I can access" user story you've listed. So have a few questions before starting so I'll group them below as how the UI connects to the backend and how to present the data.

Endpoints

  1. What will the request and response look like?
    I can add a RAML or some form of documentation for what they should look like when the UI calls the backend. Or some other solution to provide a contract for the UI to stick to and ensure it connects nicely with the backend.

  2. getAll() API Responses - What will each response body look like for the following scenarios?:

  • Array of graphs
  • One graph
  • Empty graphs array
  • Error response from API
  1. Will Admin and Any Generic User share the same API getAll() graph response body?
    i.e. the graph objects returned in response body be the same objects

Presentation

  1. Is the name of the UI App going to be Kai also? - Or Graph-as-a-Service, as named in the README.md?

  2. For each graph, what data should be presented to the User?
    e.g. Name & Description, or other data

  3. Would both Admin and Generic User view the graphs in the same way (i.e. share the same presentation logic) - or will data be hidden - or different object structure?

  4. For API errors returned, I can put a simple notification that appears at the top of the page (this can be added as another Issue later for a more enhance complex presentation) - is it ok to do this for now?

  5. CSS/styling - I've seen you've worked on another app called Gaffer alongside this one - I can copy the CSS and design similar styling as that app - let me know if this is ok too

@t11947 t11947 assigned t11947 and macenturalxl1 and unassigned t11947 May 19, 2020
@jpelbertrios
Copy link

Hi .. I am working on to setup React project for UI
I found entry point will be graph.js in lambdas, so do I need to defined

module.exports = {
entry: './path/.../graph.js'
};

in webpack.config.js ?

@d47853
Copy link
Member Author

d47853 commented Jun 8, 2020

Hi @jpelbertrios

Going forward, we're moving the project back to a Typescript layout (see #19). That file was just a placeholder for where we might put Lambda functions associated with the REST API so I don't think so.

Hopefully moving to a Typescript project will mean an easier integration between the UI and the Infrastructure as Code backend.

macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Aug 11, 2020
jpelbertrios referenced this issue in jpelbertrios/Kai Aug 11, 2020
t11947 added a commit that referenced this issue Aug 21, 2020
GH-7 Initial infrastructure/ui separation
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
Gh 07 ui develop- Merges with view graph and file drop zone for Add graph page
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
…rror-handling

Gh 07 add/view graph error handling
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
Merge branch 'develop' of github.com:gchq/Kai into gchqgh-7-ui-develop
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 pushed a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Jan 12, 2021
rocky341 added a commit to macenturalxl1/Kai that referenced this issue Jan 21, 2021
macenturalxl1 added a commit to macenturalxl1/Kai that referenced this issue Feb 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants