Skip to content

docs: populate env.example with all required variables for issue #40#78

Open
Nalwogatracy wants to merge 1 commit intobos-com:mainfrom
Nalwogatracy:fix/issue-40-env-example
Open

docs: populate env.example with all required variables for issue #40#78
Nalwogatracy wants to merge 1 commit intobos-com:mainfrom
Nalwogatracy:fix/issue-40-env-example

Conversation

@Nalwogatracy
Copy link
Copy Markdown

  • Added database, JWT, OAuth configuration
  • Added email, Redis, logging settings
  • Added API documentation and rate limiting configs

…com#40

- Added database, JWT, OAuth configuration
- Added email, Redis, logging settings
- Added API documentation and rate limiting configs
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 21, 2026

@Nalwogatracy is attempting to deploy a commit to the mukisalivingstone133-9917's projects Team on Vercel.

A member of the Team first needs to authorize it.

@RonaldRonnie
Copy link
Copy Markdown
Contributor

RonaldRonnie commented Apr 21, 2026

I will review all the PRs tonight and will update you soon tonight @Nalwogatracy
Thanks for the work done

@RonaldRonnie RonaldRonnie self-requested a review April 21, 2026 18:47
Copy link
Copy Markdown
Contributor

@RonaldRonnie RonaldRonnie left a comment

Choose a reason for hiding this comment

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

Great work @Nalwogatracy on populating the env.example file , this is very helpful for onboarding and you are just a step to success👍

A few improvements are needed before merging:

  • Please add short comments explaining each variable (as required in the issue)
  • Ensure each variable is on its own line (currently some appear concatenated)
  • Consider separating required vs optional variables for clarity
  • Clarify how these map to Spring Boot config (e.g., datasource properties)
  • Confirm that README includes instructions to copy and use this file

Once these are addressed, this should fully meet the acceptance criteria and will be ready for merging.

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.

2 participants