Skip to content

fix mongo url - #68

Merged
louisdussarps merged 5 commits into
mainfrom
fix-merge-env
Jul 21, 2025
Merged

fix mongo url#68
louisdussarps merged 5 commits into
mainfrom
fix-merge-env

Conversation

@louisdussarps

@louisdussarps louisdussarps commented Jul 18, 2025

Copy link
Copy Markdown
Contributor

Change description

Properly test if object has property in deepMerge
Do not set tls: true by default

Type of change

  • Bug fix (fixes an issue)
  • New feature (adds functionality)

Related issues

  • In deepMerge, if an object has a property set to false, deepMerge overrides it
  • It is not necesary to set tls: true as a default property

Checklists

Development

  • Lint rules pass locally
  • Application changes have been tested thoroughly
  • Automated tests covering modified code pass

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
  • reviewers assigned

@louisdussarps
louisdussarps requested a review from tmarcuzz July 18, 2025 17:31
@louisdussarps louisdussarps self-assigned this Jul 18, 2025
@louisdussarps
louisdussarps merged commit f525444 into main Jul 21, 2025
3 checks passed
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