Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add frontend <-> backend API #4

Merged
merged 16 commits into from
Feb 21, 2023
Merged

Add frontend <-> backend API #4

merged 16 commits into from
Feb 21, 2023

Conversation

jdtech3
Copy link
Contributor

@jdtech3 jdtech3 commented Feb 4, 2023

Added a /api/requirements endpoint in Express.js on the server-side, with parsing + vis-network logic into GraphComponent on client-side.

Tests were not yet implemented as this is more intended to be an initial proof-of-concept/demo and API structure/details are still very much in flux.

@jdtech3 jdtech3 added this to the MVP (Build V1) milestone Feb 4, 2023
@jdtech3 jdtech3 self-assigned this Feb 4, 2023
@@ -46,7 +48,12 @@ block content
}
},
layout: {
improvedLayout: false
hierarchical: {
enabled: false,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was used for testing hierarchical layout, hence false

@@ -1,82 +0,0 @@
var express = require('express');
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not deleted, but moved to server/src/routes/index.js (with Notion logic moved to server/src/services/notion.service.js)

@DM1122 DM1122 merged commit ce27638 into dev Feb 21, 2023
@DM1122 DM1122 deleted the frontend-backend-api branch February 21, 2023 00:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants