Skip to content

Bugfix/organization last fixes#1340

Merged
joanreyero merged 8 commits intomainfrom
bugfix/organization-last-fixes
Aug 15, 2023
Merged

Bugfix/organization last fixes#1340
joanreyero merged 8 commits intomainfrom
bugfix/organization-last-fixes

Conversation

@joanreyero
Copy link
Copy Markdown
Contributor

@joanreyero joanreyero commented Aug 15, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 9204e7c

Improved the performance and usability of the organization list feature. Optimized the database queries for organization data in organizationRepository.ts. Removed some sorting options and added a fallback display for organization descriptions in organization-list-table.vue.

🤖 Generated by Copilot at 9204e7c

We are the active ones, we defy the status quo
We optimize our queries, we make the data flow
We don't need sortable columns, we have the power to choose
We use the fallback option, we never lose

Why

How

🤖 Generated by Copilot at 9204e7c

  • Simplify queries for activity counts and active on platforms in OrganizationRepository class by using organizationId column in activities table (link, link)
  • Exclude organizations with zero activity count from query results in OrganizationRepository class (link)
  • Remove sorting functionality from name, activityCount, activeOn, identities, and createdAt columns in organization-list-table.vue component (link, link, link, link, link)
  • Use fallback option for displaying headline or description of organization in organization-list-table.vue component (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.

@joanreyero joanreyero merged commit c3907f3 into main Aug 15, 2023
@joanreyero joanreyero deleted the bugfix/organization-last-fixes branch August 15, 2023 17:29
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.

1 participant