feat: sched connector ui (CM-1085)#4139
Draft
joanagmaia wants to merge 3 commits into
Draft
Conversation
Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new Sched integration to the frontend, including platform/identity registration and a new settings-driven connect flow, with limited visibility for non-team users in the integrations list.
Changes:
- Added
schedto the sharedPlatformenum and inserted it into member identity ordering. - Implemented Sched integration UI/config (integration card, connect button, settings drawer, connected params, dropdown).
- Added frontend connect plumbing (Vuex action + API service method) and gated Sched visibility in the integrations list for team users.
Reviewed changes
Copilot reviewed 15 out of 17 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/shared/modules/platform/types/Platform.ts | Registers sched as a supported platform. |
| frontend/src/shared/modules/identities/config/identitiesOrder/member/suggestions.ts | Adds Sched to member identity suggestion ordering. |
| frontend/src/shared/modules/identities/config/identitiesOrder/member/profile.ts | Adds Sched to member profile identity ordering. |
| frontend/src/shared/modules/identities/config/identitiesOrder/member/list.ts | Adds Sched to member list identity ordering. |
| frontend/src/modules/integration/integration-store.js | Adds doSchedConnect Vuex action for creating/updating Sched integrations. |
| frontend/src/modules/integration/integration-service.js | Adds schedConnect API method hitting /sched-connect. |
| frontend/src/modules/admin/modules/integration/pages/integration-list.page.vue | Filters out sched integration for non-team users. |
| frontend/src/config/integrations/sched/config.ts | Introduces Sched integration config (components, metadata, docs link). |
| frontend/src/config/integrations/sched/components/sched-settings-drawer.vue | Adds Sched settings drawer UI and connect logic for multiple events. |
| frontend/src/config/integrations/sched/components/sched-params.vue | Displays connected Sched events summary in the integrations list item. |
| frontend/src/config/integrations/sched/components/sched-dropdown.vue | Adds dropdown entry to open Sched settings. |
| frontend/src/config/integrations/sched/components/sched-connect.vue | Adds connect CTA that opens the settings drawer. |
| frontend/src/config/integrations/index.ts | Registers Sched in the integrations registry. |
| frontend/src/config/identities/sched/config.ts | Adds identity config (image + placeholder) for Sched identities. |
| frontend/src/config/identities/index.ts | Registers Sched in the identities registry. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| <script setup lang="ts"> | ||
| import useVuelidate from '@vuelidate/core'; | ||
| import { required } from '@vuelidate/validators'; |
Comment on lines
+46
to
+56
| <app-form-item | ||
| class="mb-3" | ||
| label="Event subdomain" | ||
| :required="true" | ||
| :error-messages="{ required: 'This field is required' }" | ||
| > | ||
| <el-input | ||
| v-model="form.events[index].subdomain" | ||
| placeholder="e.g. myconference" | ||
| spellcheck="false" | ||
| /> |
Contributor
Author
|
@mbani01 initial work for Sched connector. Will validate with Jonathan and Nirav on Friday. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Type of change
JIRA ticket