Skip to content

Comments

Feature/issue 203/url global variable#236

Merged
vladislav-k1 merged 23 commits intodevfrom
feature/issue-203/url-global-variable
Mar 26, 2025
Merged

Feature/issue 203/url global variable#236
vladislav-k1 merged 23 commits intodevfrom
feature/issue-203/url-global-variable

Conversation

@vladislav-k1
Copy link
Contributor

Issue(s):

#203

Type of change: (choose required ones)

  • Refactor/Optimization

Description:

localExecute.sh can now be used to assign global constants, such as OpenAI API key and global URL that can be used to make requests to backend.
This is also used to optimise CD.

Additional context:

It is important to execute localExecute.sh before doing mvn test, mvn spring-boot:run or trying to execute the code in any other way.
If your system can't execute bash scripts for some reason, enter the required constants in application.properties, globalConstants.js and OpenAIAuthenticator.java manually.

Testing instructions:

Execute the localExecute.sh with required parameters, then do mvn test.

@vladislav-k1 vladislav-k1 added Backend Issue relates to backend code. Frontend Issue relates to frontend code. OpenAI Issue relates to OpenAI API platform. labels Mar 26, 2025
@vladislav-k1 vladislav-k1 added this to the Final Release milestone Mar 26, 2025
@vladislav-k1 vladislav-k1 self-assigned this Mar 26, 2025
Copy link
Collaborator

@GerardChabaBristol GerardChabaBristol left a comment

Choose a reason for hiding this comment

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

localexecute.sh works as intended.

Copy link
Contributor

@Siyuan106 Siyuan106 left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Collaborator

@GerardChabaBristol GerardChabaBristol left a comment

Choose a reason for hiding this comment

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

localexecute.sh works as intended.

@vladislav-k1 vladislav-k1 merged commit 603b69a into dev Mar 26, 2025
1 check passed
@vladislav-k1 vladislav-k1 deleted the feature/issue-203/url-global-variable branch March 26, 2025 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backend Issue relates to backend code. Frontend Issue relates to frontend code. OpenAI Issue relates to OpenAI API platform.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants