#287 .env generation scripts #293

Merged

c24elipe merged 2 commits from #287-ENV-generation-in-start-up-script into team_2_week_1 2026-04-14 12:49:20 +00:00

Conversation 5 Commits 2 Files changed 4 +37 -1

c24elipe commented 2026-04-14 12:26:17 +00:00

Collaborator

Copy link

Related to issue #287

Related to issue #287

c24elipe added 1 commit 2026-04-14 12:26:18 +00:00

Added scripts 3e1b01e5f9

c24elipe added the

needs review

label 2026-04-14 12:26:23 +00:00

c24elipe commented 2026-04-14 12:26:55 +00:00

Author

Collaborator

Copy link

OBS, I have not been able to test the Windows script. So only Linux has been tested

OBS, I have not been able to test the Windows script. So only Linux has been tested

c24elipe requested reviews from a22erigr, a24vinla 2026-04-14 12:27:06 +00:00

c24elipe referenced this pull request 2026-04-14 12:28:47 +00:00

ENV generation in start-up script #287

c24elipe added 1 commit 2026-04-14 12:41:11 +00:00

Fixed incorrect script call in WindowsBuild a2bfe8095c

a24vinla commented 2026-04-14 12:42:08 +00:00

Collaborator

Copy link

Review on #293 (Windows)

whats done

Tested on Windows 11

• Script run. with already existent .env file.
  • The old .env did not get over written and the app worked
• Run the app with removed .env without script
  • Connection to database was not established
• Run the script without .env file existing
  • The script creates the file copying the example .env file.
  • Was able to connect to the database.
• Inspected files
  • .env
  • CheckForEnv.sh

Needs to be done

• Nothing

Conclusion

• The code is reedy be be merged.

# Review on #293 (Windows) ## whats done Tested on Windows 11 + Script run. with already existent .env file. + The old .env did not get over written and the app worked + Run the app with removed .env without script + Connection to database was not established + Run the script without .env file existing + The script creates the file copying the example .env file. + Was able to connect to the database. + Inspected files + .env + CheckForEnv.sh ## Needs to be done + Nothing ## Conclusion + The code is reedy be be merged.

c24elipe self-assigned this 2026-04-14 12:42:31 +00:00

a22erigr commented 2026-04-14 12:45:40 +00:00

Collaborator

Copy link

Review on #293 (Linux)

whats done

Tested on Fedora 43, firefox

• Script run. with already existent .env file.
  • The old .env did not get over written and the app worked
• Run the app with removed .env without script
  • Connection to database was not established
• Run the script without .env file existing
  • The script creates the file copying the example .env file.
  • Was able to connect to the database.
• Inspected files
  • .env
  • CheckForEnv.sh

Needs to be done

• Nothing

Conclusion

• The code is reedy be be merged.

# Review on #293 (Linux) ## whats done Tested on Fedora 43, firefox + Script run. with already existent .env file. + The old .env did not get over written and the app worked + Run the app with removed .env without script + Connection to database was not established + Run the script without .env file existing + The script creates the file copying the example .env file. + Was able to connect to the database. + Inspected files + .env + CheckForEnv.sh ## Needs to be done + Nothing ## Conclusion + The code is reedy be be merged.

c24elipe commented 2026-04-14 12:49:14 +00:00

Author

Collaborator

Copy link

Good, then it gets merged

Good, then it gets merged

c24elipe merged commit 1cd0483cda into team_2_week_1 2026-04-14 12:49:20 +00:00

c24elipe deleted branch #287-ENV-generation-in-start-up-script 2026-04-14 12:49:20 +00:00

c24elipe referenced this pull request from a commit 2026-04-14 12:49:21 +00:00

Merge pull request '#287 .env generation scripts' (#293) from #287-ENV-generation-in-start-up-script into team_2_week_1

a22erigr approved these changes 2026-04-14 12:52:04 +00:00

a22erigr left a comment

Collaborator

Copy link

Se comment

Se comment

a22erigr added

reviewed

and removed

needs review

labels 2026-04-14 12:53:13 +00:00

b25marem referenced this pull request from a commit 2026-05-15 08:56:00 +00:00

Merge pull request '#287 .env generation scripts' (#293) from #287-ENV-generation-in-start-up-script into team_2_week_1

Sign in to join this conversation.

Reviewers

No reviewers

a24vinla

a22erigr

Labels

Clear labels   

_CRITICAL_

Only use this label for critical tasks that are project-scope,and must be addressed ASAP.

  

API

  

app

  

backEnd

  

Blocked, waiting for further changes

  

bug

Something is not working

  

cleanup

Issues to cleanup the codebase (restructure folders, remove deprecated or smelly code)

  

close

Issue to close

  

design

  

duplicate

This issue or pull request already exists

  

enhancement

New feature

  

feature request

  

frontEnd

  

help wanted

Need some help

  

invalid

Something is wrong

  

low priority

  

needs input

Issue needs input from product owner or customer

  

needs review

issue is in need of review

  

project documentation

  

question

More information is needed

  

research

  

reviewed

This Issue has been reviewed

  

script

  

security

use this for security-related issues

  

SQL

  

style

  

testing

  

topLevel

Reserved for top-level, project-wide issues.

  

wontfix

This won't be fixed

No labels

_CRITICAL_

API

app

backEnd

Blocked, waiting for further changes

bug

cleanup

close

design

duplicate

enhancement

feature request

frontEnd

help wanted

invalid

low priority

needs input

needs review

project documentation

question

research

reviewed

script

security

SQL

style

testing

topLevel

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

c24elipe

3 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

Andras/BoundlessFlowCampus2K!293

No description provided.