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.

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

Axios Object Usage #90

Closed
ineshbose opened this issue Jan 18, 2022 · 3 comments · Fixed by #91
Closed

Axios Object Usage #90

ineshbose opened this issue Jan 18, 2022 · 3 comments · Fixed by #91
Assignees
Projects

Comments

@ineshbose
Copy link
Owner

Is your feature request related to a problem?

Provide a clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

When the frontend makes calls to the backend using Axios, the full URL is being used everywhere.

Describe the solution you'd like

Please provide a clear and concise description of what you want to happen.

Create an axios instance using axios.create() that would use the base URL.

@ineshbose ineshbose added this to To do in Block 3 via automation Jan 18, 2022
@ineshbose
Copy link
Owner Author

/cib

Block 3 automation moved this from To do to Supervisor review Jan 18, 2022
@github-actions github-actions bot reopened this Jan 18, 2022
Block 3 automation moved this from Supervisor review to In progress Jan 18, 2022
@github-actions
Copy link

@ineshbose
Copy link
Owner Author

Really troubling issue faced - environment variables were cached (ref goatandsheep/react-native-dotenv#75). Had to clear up lot of things in order to fix this.

@github-actions github-actions bot moved this from In progress to Code review in Block 3 Jan 18, 2022
Block 3 automation moved this from Code review to Supervisor review Jan 18, 2022
@ineshbose ineshbose moved this from Supervisor review to Done in Block 3 Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Block 3
  
Done
Development

Successfully merging a pull request may close this issue.

1 participant