Skip to content

Conversation

joanagmaia
Copy link
Contributor

@joanagmaia joanagmaia commented Dec 2, 2022

Changes proposed ✍️

  • Refactor paddings, and css classes to prevent widgets from overlapping and covering the Add widget button
  • Change size of widget upon creation depending on widget type
  • Refactor FilterComponent so that styles are fixed, when first filter changes, the value resets, prefills both activity platform and activity type according to the active integrations
  • Fix Order selection on table additional settings
  • Only show Granularity selection if chart type is area, bar, or table
  • Only show Additional settings if chart type is table
  • Fix column name on widget table

Relates to #279

Checklist ✅

  • Label appropriately with Feature, Enhancement, or Bug.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated:
    • Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
    • Local backend: backend/.env.dist.local, backend/.env.dist.composed.
    • Configuration docs have been updated.
    • Team members only: update environment variables in override, staging and production env. files and trigger update config script.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@joanagmaia joanagmaia added the Bug Created by Linear-GitHub Sync label Dec 2, 2022
@joanagmaia joanagmaia requested a review from joanreyero December 2, 2022 11:58
@joanagmaia joanagmaia self-assigned this Dec 2, 2022
@joanagmaia joanagmaia merged commit 7b2efe8 into main Dec 2, 2022
@joanagmaia joanagmaia deleted the bug/reports-issues branch December 2, 2022 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant