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

Refactor: add authorization header to API request #15

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

elblogbruno
Copy link

Disclaimer: I am new to strapi.
The plugin did not work for me as I was missing a token to authenticate the request. So I did a little modification to put the token so suggestions work.

image

@cp-sumi-k
Copy link
Member

Hello @elblogbruno, Thank you for the PR.

But we shouldn't add header fields in API requests. What if there are other header requirements in the future? We can't add them individually as they are based on developer preferences.

What you can do is create a custom API, which will fetch data from your authenticated API and use that custom API here

@vikrant293
Copy link

vikrant293 commented Sep 18, 2024

Hello @cp-sumi-k can you make change in this that data which is coming from API that can only entered in tags not other such tags cann't be entered.

@cp-sumi-k
Copy link
Member

@vikrant293, You are free to create PR on this, I will review that and give insights on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants