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

chore: 128 Compliance/Create base service and client to communicate with eLicencing #2981

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

chore: 128 Compliance/Create base service and client to communicate with eLicencing #2981

wants to merge 10 commits into from

Conversation

vivanovbc
Copy link
Contributor

@vivanovbc vivanovbc commented Mar 10, 2025 •
edited
Loading

https://www.loom.com/share/454113bc5d0c488687b429d0445027ae?sid=05539611-e475-44c7-851b-78738b03ff87

Changes

  • Added base eLicensing service integration with the following components:
    • Created ELicensingService class for communicating with the eLicensing API
    • Implemented API endpoint /elicensing/test-connection to verify connectivity
    • Added test coverage for the service
  • Added environment variables for eLicensing integration:
    • ELICENSING_API_URL: URL for the eLicensing API
    • ELICENSING_AUTH_TOKEN: Authentication token for API access
  • Updated Django settings to include eLicensing configuration
  • Modified Helm templates to include eLicensing secrets in deployment

image

@vivanovbc vivanovbc self-assigned this Mar 10, 2025
nataliepiguliak
nataliepiguliak approved these changes Mar 11, 2025
View reviewed changes
Copy link
Contributor

@nataliepiguliak nataliepiguliak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@vivanovbc vivanovbc force-pushed the slava/base-elicensing-client branch from 6dad0d4 to 7ed630f Compare March 12, 2025 07:02
@vivanovbc vivanovbc force-pushed the slava/base-elicensing-client branch from b46beb5 to a61ac40 Compare March 12, 2025 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nataliepiguliak nataliepiguliak nataliepiguliak approved these changes

@Sepehr-Sobhani Sepehr-Sobhani Awaiting requested review from Sepehr-Sobhani

Assignees

@vivanovbc vivanovbc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vivanovbc @nataliepiguliak