Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

GroupEx pro integration: Consider refactoring of GroupEx integration settings forms #2390

Open
ddrozdik opened this issue Jan 22, 2021 · 0 comments
Open

GroupEx pro integration: Consider refactoring of GroupEx integration settings forms #2390

ddrozdik opened this issue Jan 22, 2021 · 0 comments
Labels
Daxko Related to Daxko integration Level: Middle Needs skilled developer to work on Severity: Minor Backlog Type: Enhancement Change request. New functionality.

Comments

@ddrozdik
Copy link
Contributor

Currently, Open Y provides a few ways of displaying data from GroupEx pro, embedded iframe, and pulling data via API, and displaying them on the Schedules page. At the moment, two different modules provide these functionalities, and both provide settings form where client id should be specified.

  1. GroupEx Pro Account Settings
    image

  2. GroupEx Pro settings
    image

In both forms, the client ID is a required setting.

It makes sense to keep only one form instead of two to simplify the configuration experience.
@ddrozdik ddrozdik added the Type: Enhancement Change request. New functionality. label Jan 22, 2021
@ddrozdik ddrozdik changed the title GroupEx pro integration: Consider refactoring of how GroupEx integration settings GroupEx pro integration: Consider refactoring of GroupEx integration settings forms Jan 22, 2021
@sarah-halby sarah-halby added Daxko Related to Daxko integration Level: Middle Needs skilled developer to work on Severity: Minor Backlog labels Jan 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Daxko Related to Daxko integration Level: Middle Needs skilled developer to work on Severity: Minor Backlog Type: Enhancement Change request. New functionality.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ddrozdik @sarah-halby