Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added logo for 'tnc' platform, added color for distribution bar in Re… #1582

Merged
merged 3 commits into from
Oct 2, 2023

Conversation

RaginiAngarka
Copy link
Contributor

@RaginiAngarka RaginiAngarka commented Sep 29, 2023

Changes proposed ✍️

What

Added logo for platform = 'tnc'
Added fix for 'Cvent' platform is not showing in the 'Activities by platform' distribution bar in Reports

🤖 Generated by Copilot at 8465690

We're adding colors to the charts of doom
We're creating files for the tnc integration
We're importing and exporting config objects
We're rocking the dashboard with our code innovation
Added logo platform = 'tnc'
https://github.com/CrowdDotDev/linux-foundation-support/issues/187

How

🤖 Generated by Copilot at 8465690

  • Add chartColor property to cvent and lfx integration configs to customize chart appearance (link, link)
  • Create new tnc integration config file with properties such as name, description, icon, chartColor, and apiUrl (link)
  • Export tnc integration config object from a separate index file for easier import (link)
  • Import and register tnc integration config in integrations-config.js file to make it available for use (link, link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@gaspergrom gaspergrom self-requested a review October 2, 2023 07:19
@gaspergrom gaspergrom merged commit c3a3759 into CrowdDotDev:crowd-linux Oct 2, 2023
5 checks passed
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.

2 participants