Content
View differences
Updated by Kiran Parajuli about 4 years ago
If the admin has not set the correct settings, the user should not see the connect button. That will avoid potential issues
* [ ] write unit tests for anything that can go wrong in `oauthRedirect`
* [ ] write unit tests for anything that can go wrong in `getOpenProjectOauthURL`
* [ ] refactor `getOpenProjectOauthURL`so that it throws an exception if any needed value is not set
* [ ] write tests for `\OCA\OpenProject\Settings\Personal::getForm`
* [ ] add check to `\OCA\OpenProject\Settings\Personal::getForm`for the case that `_getOpenProjectOauthURL_`_throws an error_
* [ ] add check to `\OCA\OpenProject\Settings\Personal::getForm`for the case, case that secret is not set (if we keep the secret in <mention class="mention" data-id="41128" data-type="work_package" data-text="#41128">#41128</mention> )
* [ ] don't show connect button in files tab, dashboard, filestab, dashboard or personal settings page if any required admin settings are not set (@kiranparajuli)
* [ ] write unit tests for anything that can go wrong in `oauthRedirect`
* [ ] write unit tests for anything that can go wrong in `getOpenProjectOauthURL`
* [ ] refactor `getOpenProjectOauthURL`so that it throws an exception if any needed value is not set
* [ ] write tests for `\OCA\OpenProject\Settings\Personal::getForm`
* [ ] add check to `\OCA\OpenProject\Settings\Personal::getForm`for the case that `_getOpenProjectOauthURL_`_throws an error_
* [ ] add check to `\OCA\OpenProject\Settings\Personal::getForm`for the case,
* [ ] don't show connect button in files tab, dashboard,