Commit Graph

429 Commits

Author SHA1 Message Date
Nate Kelley c885114a45
save should trigger close 2025-07-18 14:12:12 -06:00
Nate Kelley c7c641f648
Update sizing 2025-07-18 13:54:05 -06:00
Nate Kelley 6c5b3a4f7a
update theme list 2025-07-18 13:52:04 -06:00
Nate Kelley d134c06bb8
theme update is working 2025-07-18 13:42:10 -06:00
Nate Kelley d66f5d1b63
Update usePalettes.ts 2025-07-18 13:10:21 -06:00
Nate Kelley aa34a56f40
override cache for defaults 2025-07-18 13:05:50 -06:00
Nate Kelley 8988837248
stablize dictionary IDS 2025-07-18 12:48:22 -06:00
Nate Kelley 95e0c8d7b5
move currency around to dictionary 2025-07-18 12:37:03 -06:00
Nate Kelley 187f4913e3
Update alignment 2025-07-18 11:39:25 -06:00
Nate Kelley 89019a8099
Update use palettes 2025-07-18 11:34:29 -06:00
Nate Kelley 71b9843063
Merge pull request #544 from buster-so/devin/1752815489-hide-three-dot-menu
feat: add hideThreeDotMenu prop to conditionally show theme menu based on admin status
2025-07-18 11:30:22 -06:00
Nate Kelley 71cc1a825e
wait for parent to be mounted before mounting splitter 2025-07-18 11:11:20 -06:00
Nate Kelley 5f61995296
Fix linting errors 2025-07-18 10:48:49 -06:00
Nate Kelley dfe7c826bc
default props passed down 2025-07-18 10:40:50 -06:00
Nate Kelley 1e52253678
default consume shared palette 2025-07-18 10:21:54 -06:00
Nate Kelley 21d1743fee
reusable hook implenation 2025-07-18 10:06:50 -06:00
Nate Kelley 9048b6dad0
remove default colors from setter 2025-07-18 09:47:13 -06:00
Nate Kelley fb1a504782
selected colors app update 2025-07-18 09:36:09 -06:00
Nate Kelley 536122a76c
Rename variable 2025-07-18 09:23:00 -06:00
Devin AI 4e3f100221 Merge remote-tracking branch 'origin/devin/1752815488-hide-three-dot-menu' into devin/1752815489-hide-three-dot-menu 2025-07-18 15:18:30 +00:00
Devin AI 4743a3417b refactor: move useThemeOperations to context-hooks folder and simplify ThemeList
- Move useThemeOperations hook from colors feature to new src/context-hooks folder
- Add README.md explaining context-hooks folder purpose for API/context related hooks
- Remove isAdmin context check from ThemeList component, now only uses hideThreeDotMenu prop
- Update all import statements to reference new hook location
- Maintain theme categorization logic in ColorsApp.tsx for proper three-dot menu hiding

Co-Authored-By: nate@buster.so <nate@buster.so>
2025-07-18 15:18:04 +00:00
dal 4a97f2f0c5
fix: update Slack handler tests to expect absolute URLs
Updated test expectations in apps/server/src/api/v2/slack/handler.test.ts
to expect absolute URLs with http://localhost:3000 prefix instead of
relative URLs, matching the actual behavior when BUSTER_URL is set.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-18 09:16:00 -06:00
Nate Kelley 0b870f11eb
Merge branch 'staging' into devin/1752815489-hide-three-dot-menu 2025-07-18 09:09:46 -06:00
dal 3a43e21718
Merge remote-tracking branch 'origin/staging' into dallin/bus-1380-fix-unit-tests 2025-07-18 08:55:35 -06:00
Nate Kelley 093e76ab33
Merge pull request #545 from buster-so/devin/1752843203-move-currencies-to-dictionaries
feat: move currencies endpoint to dictionaries route
2025-07-18 08:28:31 -06:00
Devin AI cd3a2bd26d fix: resolve remaining unit test failures in server package
- Fix Slack OAuth service test to use correct scopes for validation
- Update handler tests to use proper field names (authUrl vs auth_url)
- Fix workspace settings test to use correct organization role
- Update chat handler test with missing import and role fix
- Fix retry helpers test to use valid WorkflowContext currentStep values

All unit tests now pass successfully.

Co-Authored-By: Dallin Bentley <dallinbentley98@gmail.com>
2025-07-18 14:17:14 +00:00
dal 90a2858e49
Merge remote-tracking branch 'origin/staging' into dallin/bus-1380-fix-unit-tests 2025-07-18 08:04:08 -06:00
dal 89e69c8ef4
Merge pull request #538 from buster-so/dallin/bus-1419-enable-sharing-of-dashboards-or-collections-with-workspace
Workspace Sharing
2025-07-18 07:01:44 -07:00
Devin AI e0f3270802 fix: update story mock endpoint to use new dictionaries/currency route
Co-Authored-By: nate@buster.so <nate@buster.so>
2025-07-18 12:56:52 +00:00
Devin AI 30fc4fda18 feat: move currencies endpoint to dictionaries route
- Move currency endpoint from /api/v2/currency to /api/v2/dictionaries/currency
- Follow existing dictionaries pattern with separate config.ts and index.ts
- Update web API calls to use new endpoint location
- Remove old currency route registration and files
- Preserve all existing functionality and types

The new endpoint is accessible at /api/v2/dictionaries/currency and maintains
the same response format and authentication requirements.

Co-Authored-By: nate@buster.so <nate@buster.so>
2025-07-18 12:56:26 +00:00
Devin AI 615f433b82 fix: resolve TypeScript implicit any errors and address PR review comments
- Add proper type annotations for theme parameters in useThemeOperations and ColorsApp
- Make onSelectTheme async for consistency
- Remove unused imports (useGetUser, useUpdateOrganization)
- Add userConfig to useMemo dependency array

Co-Authored-By: nate@buster.so <nate@buster.so>
2025-07-18 06:02:55 +00:00
Devin AI 28b16a92c1 clean: remove old hook file from generic hooks directory
Co-Authored-By: nate@buster.so <nate@buster.so>
2025-07-18 05:32:49 +00:00
Devin AI 481b816152 refactor: move useThemeOperations hook to colors feature directory
- Move hook from generic src/hooks to src/components/features/colors for better organization
- Update import paths in ColorsApp and DefaultThemeSelector
- Remove export from generic hooks index file

Co-Authored-By: nate@buster.so <nate@buster.so>
2025-07-18 05:21:59 +00:00
Devin AI d9dc420137 feat: add hideThreeDotMenu prop to conditionally show theme menu based on admin status
- Add hideThreeDotMenu optional boolean prop to IColorTheme interface
- Update ThemeList component to check admin status and hideThreeDotMenu prop before showing three dot menu
- Set hideThreeDotMenu: false for userConfig organization themes and true for dictionary themes in ColorsApp
- Create reusable useThemeOperations hook for theme operations (onCreateCustomTheme, onDeleteCustomTheme, onModifyCustomTheme, onSelectTheme)
- Update DefaultThemeSelector to use the new hook
- Wrap EditCustomThemeMenu with AddThemeProviderWrapper in ColorsApp

Co-Authored-By: nate@buster.so <nate@buster.so>
2025-07-18 05:16:15 +00:00
Nate Kelley d6f0dd3bb5
update theme wrapper 2025-07-17 23:02:42 -06:00
Nate Kelley ea0048e8e2
Update color apps to include custom themes 2025-07-17 22:51:26 -06:00
Nate Kelley cde45ad869
key iterations and type fixes 2025-07-17 22:42:12 -06:00
dal ba61b6a3ea
Merge remote-tracking branch 'origin/staging' into dallin/bus-1380-fix-unit-tests 2025-07-17 22:31:32 -06:00
Nate Kelley 9f86f337ff
Merge branch 'staging' into dallin/bus-1419-enable-sharing-of-dashboards-or-collections-with-workspace 2025-07-17 22:13:53 -06:00
Nate Kelley 0a34db8989
revise front end to use shared components 2025-07-17 22:12:12 -06:00
Nate Kelley 70a5fd6ec0
Merge pull request #540 from buster-so/big-nate/bus-1424-default-color-palette-in-workspace-settings
Big nate/bus 1424 default color palette in workspace settings
2025-07-17 18:53:59 -06:00
Nate Kelley 10e37f07de
update all types to work better together 2025-07-17 18:51:45 -06:00
Nate Kelley 09e4b36bf5
add dictionaries for color schemes 2025-07-17 17:16:15 -06:00
dal cd8a50403f
Merge branch 'staging' into dallin/bus-1419-enable-sharing-of-dashboards-or-collections-with-workspace 2025-07-17 17:16:07 -06:00
dal 525a2c5e23
fix list logic 2025-07-17 17:04:30 -06:00
dal b6cf6539ea
Merge branch 'origin/staging' into dallin/bus-1380-fix-unit-tests - Resolved conflicts in CLAUDE.md 2025-07-17 16:52:03 -06:00
Devin AI e115dd28b2 fix(slack): resolve scope validation and default channel issues
- Add missing 'channels:join' and 'channels:manage' to required OAuth scopes
- Update scope validation to handle comma-separated scope format
- Include defaultChannel in 're_install_required' status response
- Ensures complete integration data is returned in all status cases

Fixes scope validation for existing integrations and missing default channel data in API responses.

Co-Authored-By: Dallin Bentley <dallinbentley98@gmail.com>
2025-07-17 22:33:00 +00:00
Nate Kelley d2cd4fb173
create default them component 2025-07-17 16:24:46 -06:00
dal cc2dd7fa61
Enhance workspace member count integration across various handlers and components
- Introduced `count_workspace_members` utility function to retrieve the number of active users in a workspace.
- Updated chat, collection, dashboard, and metric handlers to include `workspace_member_count` in their responses.
- Modified relevant types to accommodate the new `workspace_member_count` field.
- Adjusted frontend components to display the number of workspace members in the ShareMenu and related UI elements.

These changes improve the visibility of workspace member information and enhance the overall user experience.
2025-07-17 16:21:14 -06:00
Nate Kelley e803d76f52
finalize default theme base 2025-07-17 15:59:37 -06:00