-
Notifications
You must be signed in to change notification settings - Fork 3
feat(cms/api): add basic API client #238
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
Open
Varun-Sethu
wants to merge
41
commits into
main
Choose a base branch
from
varun/APIClient
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
736dd1f
Adding basic API client
Varun-Sethu d8bb2cf
added API type assertations
Varun-Sethu f64cae6
Merge branch 'main' of github.com:csesoc/cms.csesoc.unsw.edu.au into …
Varun-Sethu 4a3207a
fixing expect message
Varun-Sethu a9394cf
updating workflow to support integration tests
Varun-Sethu 7567dc2
undoing changes to ghost exporter
Varun-Sethu efcf807
undoing changes to ghost exporter
Varun-Sethu ae353a3
oh yaml gods please work
Varun-Sethu 72652d8
i smite thee yaml gods
Varun-Sethu 720a94f
all i know is pain and suffering
Varun-Sethu 95794e3
please jsut run the stupid backend tests
Varun-Sethu 167b3fc
i think this works now
Varun-Sethu db91d04
tests pass (locally)
Varun-Sethu 9045648
i guess not
Varun-Sethu dfe482d
:(
Varun-Sethu 4c7f9b6
please just just work please pleaaase
Varun-Sethu b44edd2
i dont know how to title these commits anymore
Varun-Sethu f6c0ba4
i dont know how to title these commits anymore
Varun-Sethu 2da112b
upgrading ubuntu version
Varun-Sethu 2e0c432
upgrading ubuntu version
Varun-Sethu 18066b3
i think this might be it! :D
Varun-Sethu e1315d5
adding CI configuration
Varun-Sethu e82e62b
fixing client config
Varun-Sethu 1abd37a
throws hands in air
Varun-Sethu 92a6c76
i reaaallly hope this works
Varun-Sethu 8374a04
hopefully this works again
Varun-Sethu dc36f41
hopefully this works again
Varun-Sethu aede6b6
printf debugging begins
Varun-Sethu 1f5189d
capturing logs
Varun-Sethu 0e26cdb
im actually going insane
Varun-Sethu d85c44c
updating config again
Varun-Sethu 127f5dc
updating config again
Varun-Sethu 993b580
capturing more debugging info
Varun-Sethu 234fd8e
testing docker network
Varun-Sethu 01ef76d
testing docker network
Varun-Sethu 75b8261
backend logging
Varun-Sethu e34370d
backend logging
Varun-Sethu 5ec1dec
Update frontend/src/packages/api/tests/filesystemConsistency.test.ts
Varun-Sethu 4abce87
Update frontend/src/packages/api/types/general.ts
Varun-Sethu 41c6103
Apply suggestions from code review
Varun-Sethu 1bc173a
Apply suggestions from code review
Varun-Sethu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,5 +30,7 @@ func main() { | |
handler := cors.Default().Handler(mux) | ||
handler = c.Handler(handler) | ||
|
||
log.Print("CMS Go backend starting on port :8080 :D.") | ||
log.Print("Amongus.") | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🍣 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ඞ |
||
log.Fatal(http.ListenAndServe(":8080", handler)) | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
BACKEND_URI=http://localhost:8080/ | ||
FRONTEND_URI=http://localhost:3000/ | ||
PG_USER=postgres | ||
PG_PASSWORD=postgres | ||
PG_DB=test_db | ||
PG_PORT=5432 | ||
PG_HOST=db:5432 | ||
E2E_MODE=github |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uh sorry so just to clarify but how is the docker compose build part running any sort of tests? And should should the lines at the bottom be commented out?
cc @sachk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
smh i pointed this out and now jared is STEALING the credit - more ABUSE by csesoc execs :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The bottom two lines were commented for testing reasons (they should not be commented out), the entire PR is still blocked by the weird networking issue 😢