Skip to content

Merge into Main #446

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

Merged
merged 81 commits into from
Apr 25, 2025
Merged

Merge into Main #446

merged 81 commits into from
Apr 25, 2025

Conversation

Marfuen
Copy link
Contributor

@Marfuen Marfuen commented Apr 24, 2025

What does this PR do?

  • Fixes #XXXX (GitHub issue number)
  • Fixes COMP-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description)

Visual Demo (For contributors especially)

A visual demonstration is strongly recommended, for both the original and new change (video / image - any one).

Video Demo (if applicable):

  • Show screen recordings of the issue or feature.
  • Demonstrate how to reproduce the issue, the behavior before and after the change.

Image Demo (if applicable):

  • Add side-by-side screenshots of the original and updated change.
  • Highlight any significant change(s).

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Are there environment variables that should be set?
  • What are the minimal test data to have?
  • What is expected (happy path) to have (input and output)?
  • Any other important info that could help to test that PR

Checklist

  • I haven't read the contributing guide
  • My code doesn't follow the style guidelines of this project
  • I haven't commented my code, particularly in hard-to-understand areas
  • I haven't checked if my changes generate no new warnings

Marfuen and others added 18 commits April 16, 2025 13:21
…-create-tasks-table-and-move-evidence-to-be-tasks
- Moved S3 client configuration to a new shared module for better maintainability.
- Updated uploadFile, deleteCommentAttachment, deleteTaskAttachment, getCommentAttachmentUrl, and getTaskAttachmentUrl actions to use the shared S3 client and bucket name.
- Removed redundant environment variable checks from individual action files.
- Enhanced error handling for missing AWS credentials in the shared module.
…ing adjustments

- Changed border radius from rounded-md to rounded-sm in various components including code blocks, cards, and calendars.
- Updated CSS variables for input styling to enhance visual consistency.
- Refactored alert dialog and dropdown menu components for improved structure and styling.
- Adjusted tooltip and popover components for better alignment with design standards.
- Updated various components to use rounded-sm instead of rounded-md for consistent styling.
- Adjusted layout and styling in components such as EmployeesListSkeleton, EvidenceDetails, and FileCard.
- Enhanced visual consistency in data tables and integration cards by applying the new border radius.
…-create-tasks-table-and-move-evidence-to-be-tasks
Copy link

vercel bot commented Apr 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2025 4:35pm
comp-portal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2025 4:35pm

@vercel vercel bot temporarily deployed to Preview – comp-portal April 24, 2025 14:25 Inactive
@vercel vercel bot temporarily deployed to Preview – app April 24, 2025 14:25 Inactive
- Introduced new rules for better-auth regarding server-side and client-side usage of auth methods.
- Added `MemberRow` and `PendingInvitationRow` components for improved member management UI.
- Refactored `TeamMembers` and `TeamMembersClient` to streamline member and invitation handling.
- Removed deprecated `InviteMemberForm` and `PendingInvitations` components.
- Updated layout and styling for better responsiveness and user experience.
…esponsiveness

- Added padding to the main content area in the dashboard layout for better spacing.
- Updated header component to use a softer background and added a border for better visual separation.
- Refined styles in main menu and chat components to enhance user interaction and aesthetics.
- Adjusted input and sheet components to use rounded-sm for a consistent design language.
- Updated TabsList to use a consistent height and added rounded-lg for better aesthetics.
- Refined TabsTrigger and TabsContent with rounded-md and focus-visible styles for enhanced accessibility and visual feedback.
- Adjusted padding and margin for a more cohesive design across tab components.
… menu

- Modified the MemberRow component to display the dropdown menu only if the current role is not "owner".
- Improved the structure of the dropdown menu items for changing roles and removing members, ensuring better accessibility and user experience.
- Enhanced the overall logic for handling role changes and member removal actions.
- Added bottom padding to the main content area in the dashboard layout for better spacing.
- Refactored ChecklistItem component to improve structure and visual feedback, including updated icons and layout adjustments.
- Updated OnboardingProgress component to enhance the display of onboarding steps with improved text and layout.
- Adjusted main menu item height and spacing for a more consistent design.
- Added font smoothing to global styles for improved text rendering.
Marfuen and others added 4 commits April 25, 2025 12:01
- Added `Layout` component for employee management, ensuring organization context and onboarding for users without employees.
- Introduced `Loading` component for better user experience during data fetching.
- Created `EmployeesOverview` component to display employee statistics and training video completion.
- Developed `EmployeeCompletionChart` for visual representation of employee task completion metrics.
- Enhanced `AppOnboarding` component to support dynamic links for onboarding actions.
@vercel vercel bot temporarily deployed to Preview – comp-portal April 25, 2025 16:08 Inactive
- Added `descriptionKey` to selectable roles in `MultiRoleCombobox` for improved clarity on role functionalities.
- Updated localization file to include descriptions for each role, enhancing user understanding of permissions.
- Adjusted UI layout in `MultiRoleCombobox` to display role descriptions below the role labels, ensuring a more informative user experience.
claudfuen and others added 7 commits April 25, 2025 12:11
- Modified the `fetchAdminUsers` function to allow fetching users with emails ending in both "@trycomp.ai" and "@securis360.com".
- Updated the comment header for clarity on the supported email domains.
…uration

- Updated middleware to exclude the 'research' route from authentication checks.
- Refactored `researchVendorAction` to utilize the new `researchVendor` task for improved vendor data scraping.
- Simplified the vendor creation process in `CreateVendorForm` by awaiting the execution of the `researchVendor` action.
- Introduced a new `research.ts` file to define the schema and logic for the vendor research task, enhancing modularity and maintainability.
- Introduced a new admin menu item in the `MainMenu` component for users with emails ending in "@trycomp.ai".
- Updated localization file to include a label for the admin menu item, enhancing user experience and access control.
feat: implement search functionality for global vendors
…yout

- Enhanced button styles to include focus-visible ring and whitespace handling for better accessibility.
- Adjusted class names for consistent spacing and SVG pointer events, ensuring a more user-friendly experience.
…sbetterauth

Claudio/comp 82 fix permissionsbetterauth
carhartlewis and others added 2 commits April 25, 2025 17:27
- Adjusted class names in the button component for better alignment and spacing.
- Ensured consistent handling of whitespace and SVG pointer events for enhanced usability.
- Implemented a switch statement to dynamically render role descriptions based on the selected role in the `MultiRoleCombobox`.
- Enhanced localization by utilizing specific keys for each role's description, improving user understanding of role functionalities.
@vercel vercel bot temporarily deployed to Preview – comp-portal April 25, 2025 16:30 Inactive
@vercel vercel bot temporarily deployed to Preview – app April 25, 2025 16:32 Inactive
@claudfuen claudfuen marked this pull request as ready for review April 25, 2025 16:36
@claudfuen claudfuen merged commit cd2fe6d into main Apr 25, 2025
5 of 9 checks passed
@claudfuen claudfuen deleted the dev branch April 25, 2025 16:36
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.

3 participants