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.

Sign up for GitHub

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

Jump to bottom

Replacing the rendering view template in backend with the JSON response. #130

Open
TJSidhu1 opened this issue Apr 11, 2023 · 1 comment
Open

Replacing the rendering view template in backend with the JSON response. #130

TJSidhu1 opened this issue Apr 11, 2023 · 1 comment
Assignees
@briswells @MSS-Shalavritha @TJSidhu1
Labels
backend concerns backend/API presentation

Comments

@TJSidhu1
Copy link
Contributor

TJSidhu1 commented Apr 11, 2023 •
edited by AbhinavReddy-Dev
Loading

Made changes to API endpoints send json response instead of rendering a template, since we started using React UI.
@briswells , we are taking up this issue, keeping you in the loop.
@hardikpatil hardikpatil added the backend concerns backend/API presentation label Apr 14, 2023
TJSidhu1 added a commit that referenced this issue Apr 20, 2023
@TJSidhu1
Json response changes to the files index.js and feed.js along with ch…
1d21a64 
…anges to bypass authentication Fixes #130
TJSidhu1 added a commit that referenced this issue Apr 20, 2023
@TJSidhu1
Json response changes to the files index.js and feed.js along with ch…
9feae4d 
…anges to bypass authentication Fixes #130
MSS-Shalavritha added a commit that referenced this issue Apr 24, 2023
@MSS-Shalavritha
Fixes #130: Updated sale.js for json response
3e3a419
@kbuffardi
Copy link
Contributor

This issue needs more documentation. It isn't clear what the definition of done is.

MSS-Shalavritha added a commit that referenced this issue Apr 26, 2023
@MSS-Shalavritha
Updated users.js for json response, Fixes #130
12a499d
MSS-Shalavritha added a commit that referenced this issue Apr 26, 2023
@MSS-Shalavritha
Revert "Updated users.js for json response, Fixes #130"
a355ecc 
This reverts commit 12a499d.
MSS-Shalavritha added a commit that referenced this issue Apr 26, 2023
@MSS-Shalavritha
Updated users.js for new user sign up and user already exists, Fixes #…
a31157e 
…130
TJSidhu1 added a commit that referenced this issue Apr 26, 2023
@TJSidhu1
Revert "Json response changes to the files index.js and feed.js along…
9d1cb22 
… with changes to bypass authentication Fixes #130"

This reverts commit 1d21a64.
MSS-Shalavritha added a commit that referenced this issue Apr 27, 2023
@MSS-Shalavritha
Updated users.js file to appropriate json response, Fixes #130
d4bb17c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@briswells briswells

@MSS-Shalavritha MSS-Shalavritha

@TJSidhu1 TJSidhu1

Labels
backend concerns backend/API presentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kbuffardi @hardikpatil @briswells @MSS-Shalavritha @TJSidhu1