old
/
LinkList
2
0
Fork 0
Code Issues 3 Pull Requests Projects Releases Wiki Activity
Labels Milestones

Add frontend and backend integration example #13

Merged
a3thakra merged 0 commits from adi-add-integration-example into main 2 years ago
Conversation 4 Commits N/A Files Changed N/A
a3thakra commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

This MR adds an example component that interacts with the backend. This can serve as an example of how to integrate with the rest (pun intended) of the app.

This is what the example component looks like:

example

I think we should also scope all backend routes to /api. This wouldn't require any change from the backend, but when deploying, we just need to make sure that the base url for the backend is "https://csclub.uwaterloo.ca/links/api". I also updated the client-server integration diagram to reflect this change. Here's the diagram:

client-server-interaction.svg

CC @n3parikh for the backend change

This MR adds an example component that interacts with the backend. This can serve as an example of how to integrate with the **rest** (pun intended) of the app. This is what the example component looks like: ![example](/uploads/9b3844fd66e204a43aa6006c3a930cc9/example.gif) I think we should also scope all backend routes to `/api`. This wouldn't require any change from the backend, but when deploying, we just need to make sure that the base url for the backend is "https://csclub.uwaterloo.ca/links/api". I also updated the client-server integration diagram to reflect this change. Here's the diagram: ![client-server-interaction.svg](/uploads/d4c3fb8a660f8cc133fef0b939ee19bc/client-server-interaction.svg) CC @n3parikh for the backend change
q63xu (Migrated from git.uwaterloo.ca) approved these changes 2 years ago
a3thakra commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

changed the description

changed the description
a3thakra commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

added 6 commits

  • 3025ffce - 1 commit from branch main
  • 04ee70cc - Autoformat JS as well
  • 684ed192 - Update backend greeting
  • 8e78eb46 - Use /api for the backend
  • 20998ba0 - Add integration example
  • 4ee3b6ba - Scope backend to /api

Compare with previous version

added 6 commits <ul><li>3025ffce - 1 commit from branch <code>main</code></li><li>04ee70cc - Autoformat JS as well</li><li>684ed192 - Update backend greeting</li><li>8e78eb46 - Use /api for the backend</li><li>20998ba0 - Add integration example</li><li>4ee3b6ba - Scope backend to /api</li></ul> [Compare with previous version](/csc/linktree/-/merge_requests/8/diffs?diff_id=51181&start_sha=9753568e00a5f3d27c7d1d87bd69dd98cc740e53)
q63xu commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

Thank you for setting this up for our dev env 💯

Thank you for setting this up for our dev env :100:
q63xu commented 2 years ago (Migrated from git.uwaterloo.ca)
Owner

approved this merge request

approved this merge request

Reviewers

q63xu
The pull request has been merged.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
q63xu
Labels
Apply labels
Clear labels
deployment
No Label deployment
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: old/LinkList#13
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'adi-add-integration-example'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes