Skip to content

Conversation

@mfarhan943
Copy link
Contributor

@mfarhan943 mfarhan943 commented Nov 19, 2024

Related Ticket:
#76

Related PR:
openedx/enterprise-subsidy#319

Tasks

  • Add the service in docker-compose.yml file, follow docker-compose.yml file from the source repo.
  • Add dev.shell make command if some extra env variables need to be set
  • Add an entry in check.sh if needed
  • Mount volume in docker-compose-host.yml file
  • Add entry in options.mk file
  • Create provision-IDA_NAME.sh file if needed. Also, add an entry in provision.sh file.
  • Create table in provision.sql file if needed
  • Add an entry in repo.sh file.
  • Add an entry in service_list.rst file.

Copy link
Contributor

@iamsobanjaved iamsobanjaved left a comment

Choose a reason for hiding this comment

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

Just a few suggestions, rest looks good.

@mfarhan943 mfarhan943 force-pushed the farhan/add-enterprise-subsidy branch from 4835844 to 731be3f Compare November 26, 2024 10:36
@mfarhan943
Copy link
Contributor Author

Just a few suggestions, rest looks good.

I have applied the suggestions.

@iamsobanjaved iamsobanjaved linked an issue Feb 11, 2025 that may be closed by this pull request
@iamsobanjaved iamsobanjaved force-pushed the farhan/add-enterprise-subsidy branch 4 times, most recently from af0f32f to 0eb6bd9 Compare February 18, 2025 11:56
@iamsobanjaved iamsobanjaved force-pushed the farhan/add-enterprise-subsidy branch from ab2dcf5 to 9882734 Compare March 28, 2025 15:05
@iamsobanjaved iamsobanjaved requested a review from UsamaSadiq April 3, 2025 14:18
@iamsobanjaved iamsobanjaved merged commit 69337bc into master Apr 4, 2025
14 checks passed
@iamsobanjaved iamsobanjaved deleted the farhan/add-enterprise-subsidy branch April 4, 2025 19:05
@iamsobanjaved iamsobanjaved restored the farhan/add-enterprise-subsidy branch May 16, 2025 06:51
@iamsobanjaved iamsobanjaved deleted the farhan/add-enterprise-subsidy branch May 16, 2025 07:41
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.

Add openedx/enterprise-subsidy to devstack

5 participants