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

Trademark Search API #1858

Merged
merged 13 commits into from
Mar 1, 2024
Merged

Trademark Search API #1858

merged 13 commits into from
Mar 1, 2024

Conversation

HanilJain
Copy link
Contributor

New Trademark Search API

Due to Multiple Merge conflicts and linting issue I have to create this PR to replace #1818

  • it consistent all the suggestion and reviews and it's changes
  • no merge conflicts
  • no pre-commit issues
  • features similar to the previous PR

@HanilJain HanilJain mentioned this pull request Feb 29, 2024
@HanilJain
Copy link
Contributor Author

@DonnieBLT @arkid15r @AtmegaBuzz

I have closed the PR #1818 due to complicated merge conflicts issues
and integrated all the changes you have suggested in this PR.
you can please review.

Also regarding the API KEY
I'll create one and share you personally on slack.
but there is limit to free API on it 500 / month and 1000 requests per hour.

website/views.py Outdated Show resolved Hide resolved
website/views.py Outdated Show resolved Hide resolved
@DonnieBLT DonnieBLT enabled auto-merge (squash) March 1, 2024 04:48
@DonnieBLT DonnieBLT merged commit 2adfb56 into OWASP-BLT:main Mar 1, 2024
7 checks passed
@HanilJain HanilJain deleted the issue-998-new branch March 1, 2024 06:10
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.

None yet

3 participants