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

Fixed FE-BE connection bug #251 #265

Merged
merged 25 commits into from
Jun 11, 2024
Merged

Fixed FE-BE connection bug #251 #265

merged 25 commits into from
Jun 11, 2024

Conversation

Corgam
Copy link
Collaborator

@Corgam Corgam commented Jun 11, 2024

Fixed the problem with URLs being different on specific deployment environments.
Deleted the labeling of docker images, now we have one for each of the deployment envs.

dancingsushii and others added 20 commits June 5, 2024 08:27
@dancingsushii
Feature template changed
8371748 
Signed-off-by: dancingsushii <[email protected]>
@dancingsushii
Deliverables for week 6: planning doc and feature board
6e6b4df 
Signed-off-by: dancingsushii <[email protected]>
@op-hub
Create user-documents.pdf
8cac82a 
Signed-off-by: op-hub <[email protected]>
@CelineMP
Create user-documents.pdf (#249)
432cfce
@op-hub
Add files via upload
74c86f6 
Signed-off-by: op-hub <[email protected]>
@CelineMP
Add files via upload (#250)
8b50955
@CelineMP
Merge branch 'product_owners_deliverables'
bbed5c9
@Corgam
Sprint release 06 (#252)
8d0ee2c
@Corgam
Removed unecessary file
863bde0 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Product owners deliverables (#246)
1009ab5
@Battlemech
Add files via upload
4ce6988 
Signed-off-by: Erik Fischer <[email protected]>
@Corgam
Deliverables sprint 6, pls merge :) (#254)
390a0f0
@Corgam
Merge branch 'main' into CI-CD
4aed288
@Corgam
Fix FE url endpoint changing for each deployment environment part1
3b51d59 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Revert "Fix FE url endpoint changing for each deployment environment …
d611252 
…part1"

This reverts commit 3b51d59.
@Corgam
Finished the API url changing based on the deployment env.
a63545c 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Further work on FE deployment
60d9a4d 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Fixed sed delimiter
d571424 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Clean up of the env files
4030e61 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam
Removed multiple labels from docker images
659db49 
Signed-off-by: Emil Balitzki <[email protected]>
@Corgam Corgam requested a review from Lucas-Nan June 11, 2024 14:49
@Corgam Corgam marked this pull request as draft June 11, 2024 14:49
@Corgam Corgam marked this pull request as ready for review June 11, 2024 15:26
Lucas-Nan
Lucas-Nan approved these changes Jun 11, 2024
View reviewed changes
Copy link
Contributor

@Lucas-Nan Lucas-Nan left a comment

Choose a reason for hiding this comment

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

local deployment works! test env tested by emil! not working is house footprints but this is extra

@Corgam Corgam merged commit 64a9902 into sprint-release Jun 11, 2024
18 checks passed
@Corgam Corgam mentioned this pull request Jun 11, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Lucas-Nan Lucas-Nan Lucas-Nan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Corgam @Lucas-Nan @dancingsushii @op-hub @CelineMP @Battlemech