Added missing checking if Settings DB exists - Githubissues

apinf / platform

Apinf - Open source API management platform with multi proxy and protocol support

https://apinf.com/

European Union Public License 1.1

74 stars 35 forks source link

Added missing checking if Settings DB exists #3643

Closed matleppa closed 5 years ago

matleppa commented 5 years ago

Closes #3642

Changes

added checking if Settings exists when Sign-in page is opened
corrected error message

Testing:

logout from Apinf
rename Settings DB
refresh page

Expected results:

no exceptions

Erroneous results:
Exception with text: "Update mail configuration:"

Developer checklist

This checklist is to be completed by the PR developer:

[ ] Alternative solutions were compared/discussed before writing code
- [ ] trade-offs with this solution are considered acceptable
[ ] Code in this PR adheres to the project styleguide
[ ] This pull request does not decrease project test coverage
[ ] If the code changes existing database collection(s), migration has been written
[ ] If UI texts are added or changed, all texts are internationalized

Reviewer checklist

Reviewed by: @username1

This list is to be completed by the pull request reviewer:

[ ] Code works as described/expected
[ ] Code seems to be error free
- [ ] no browser console errors visible
- [ ] no server console errors visible
- [ ] passes CI build
[ ] Code is written in a way that promotes maintainability
- [ ] easy to understand
- [ ] well organized
- [ ] follows project coding standards and conventions
- [ ] well documented