Skip to content

Conversation

@matheusbsilva137
Copy link
Contributor

Proposed changes (including videos or screenshots)

  • Apps are able to define required settings. These settings should not be left blank by the user and an error will be thrown and shown in the interface if an user attempts to save changes in the app details page leaving any required fields blank;
    prt_screen_required_app_settings_warning

  • A sign (*) is added to the label of app settings' fields that are required so as to highlight the fields which must not be left blank.
    prt_screen_required_app_settings

Issue(s)

Task - ClickUp, which is related to the PR #367 on the Apps-Engine's side.

Steps to test or reproduce

The setting fields for each app are listed in the App details screen, under the Apps section in the administration area (Administration > Apps > App's Name).

Further comments

@ggazzo ggazzo merged commit 8cedbe2 into develop Feb 19, 2021
@ggazzo ggazzo deleted the required-settings-app-ui branch February 19, 2021 03:44
@sampaiodiego sampaiodiego mentioned this pull request Feb 28, 2021
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.

4 participants