-
Notifications
You must be signed in to change notification settings - Fork 0
Initial repository setup with listing information and reservation request accept functionality #258
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
Open
Copilot
wants to merge
121
commits into
main
Choose a base branch
from
copilot/sub-pr-257
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
121 commits
Select commit
Hold shift + click to select a range
7675212
refactor: standardize import formatting and improve code readability …
dani-vaibhav 0404a49
Initial plan
Copilot 815cd1f
feat: implement dynamic action buttons based on reservation request s…
Copilot 8eba1a5
feat: implement acceptReservationRequest mutation and integrate with UI
Copilot 1f8a9c4
feat: enhance MyListingsDashboard and RequestsTable with sharerId pro…
dani-vaibhav c3b3d3c
feat: enhance reservation request handling with improved state mappin…
dani-vaibhav fde5275
feat: update MyListingsDashboard to use sharerId for requests count a…
dani-vaibhav 9618ccd
Dummy change
dani-vaibhav c43eaa6
Dummy commit 2
dani-vaibhav c41d588
Add unit tests for ItemListingRepository and ItemListingUnitOfWork
dani-vaibhav 160864d
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 9bd8d4c
Add TypeScript binary scripts for Twilio service
dani-vaibhav 4a13019
fix: address critical code review issues
Copilot c2b8856
refactor: align all layers to use Requested and Accepted terminology
Copilot 726b409
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 7e6dc74
refactor: clean up imports and improve code readability in MyListings…
dani-vaibhav 52fd55e
feat: implement auto-rejection of overlapping reservation requests up…
dani-vaibhav c6efd84
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 6331956
feat: add automatic conversation creation on reservation acceptance
dani-vaibhav f94f382
feat: add queryOverlapByListingIdAndReservationPeriod method to reser…
dani-vaibhav b3d8769
Add '@sthrift/domain' package to pnpm-lock.yaml
dani-vaibhav 7688345
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 72243ca
Update dummy image URLs in ListingRequestDomainShape and improve imag…
dani-vaibhav 0b1c737
Enhance ListingRequestDomainShape with thumbnailUrl and images proper…
dani-vaibhav 6bc709f
refactor: address PR review comments - remove out-of-scope features
dani-vaibhav f8cb220
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav dfc2c69
refactor: improve data loading and error handling in MyListingsDashbo…
dani-vaibhav 7a6f87f
chore: update pnpm-lock.yaml after removing event-handler dependencies
dani-vaibhav 3eab4f7
fix: resolve security vulnerabilities with pnpm audit --fix
dani-vaibhav 2c555ad
fix: add pnpm overrides to resolve security vulnerabilities
dani-vaibhav 9eb1054
chore: add node-forge override to pnpm-workspace.yaml
dani-vaibhav 8a5ab89
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 5a89483
fix: update axios version to resolve compatibility issues
dani-vaibhav 3ed4dd8
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav d9371f1
fix: update sharer ObjectId and enhance population logic in reservati…
dani-vaibhav a84e139
fix: address code review feedback - remove debug logs and node_modules
Copilot d075a2d
fix: replace cpx with rsync for Node.js v22 compatibility
arif-u-ahmed 37cb3eb
feat: implement reservation request update functionality with state m…
dani-vaibhav 8f6f98f
feat: add update functionality to reservation request service and res…
dani-vaibhav 339e545
refactor: remove accept reservation request functionality
dani-vaibhav 9658bda
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav bb62191
fix: remove state mappings and fix typo in ui-components package.json
Copilot f5d863f
refactor: move business logic from update service to domain layer
Copilot e792c16
Updated package.json as per the current main branch
dani-vaibhav b81d582
fix: align listing getter pattern with codebase convention
Copilot fdd05d1
fix: update object property access in resolver helper and remove unus…
dani-vaibhav 5dd21ba
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav adffa51
fix : Success Toast Message after accepting the request
dani-vaibhav fc3f483
fix: update pnpm-lock.yaml to remove deprecated node-forge version an…
dani-vaibhav 516c64c
fix : Build issue
dani-vaibhav 7fc22a5
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 18c374d
fix : build issue
dani-vaibhav 5f55249
chore: update express and @apollo/server versions in pnpm-lock.yaml
dani-vaibhav 671e4f8
fix : Build error issues
dani-vaibhav 7696bb1
fix: Build error issues
dani-vaibhav e044ff1
fix : Build issue
dani-vaibhav c280c5c
Reset package.jsons to main version and update pnpm lock
jasonmorais ebfb97e
updated lock file
jasonmorais 01ddd91
fix vulernabilities
jasonmorais f8563b5
fixed dependency in types and actual apckage
jasonmorais 019b9d2
refactor: Update user and listing fields to use object structure in a…
dani-vaibhav d50efb3
feat: Add scenarios for fallback behavior in PopulateUserFromField an…
dani-vaibhav fcaafc4
Merged main
dani-vaibhav 3273b4e
fix: Update pnpm lock.yaml
dani-vaibhav 2f6f62b
refactor: Improve population logic in ReservationRequest domain adapt…
dani-vaibhav a41284d
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 6dd5645
fix: Update status handling in getStatusTagClass and getActionButtons…
dani-vaibhav 186fda9
fix: Update request acceptance logic and add tests for status helpers
dani-vaibhav 23d05bf
feat: Implement comprehensive update feature and tests for reservatio…
dani-vaibhav 6e1f839
fix: implement action handler stubs and remove unused sharerEmail field
Copilot 6a7666d
fix: remove sharerEmail from reservation request acceptance logic
dani-vaibhav 050baf5
feat: Add scenarios for accepting reservation requests in resolvers
dani-vaibhav bdce90b
fix: address @jasonmorais code review feedback
Copilot bed90c3
fix: update request handling functions to prepare for future mutations
dani-vaibhav b587379
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 07ab9be
feat: add message button for closed requests in action buttons
dani-vaibhav a87af97
refactor: remove export keyword from interface declarations in listin…
dani-vaibhav 7dbe837
feat: enhance request handling and button rendering for various reque…
dani-vaibhav 4cc12ac
feat: add mocks for reservation request and user in tests
dani-vaibhav 2a007a0
feat: enable query depth limit validation in Apollo Server
dani-vaibhav d8f5760
refactor: remove graphql-depth-limit dependency and related code to f…
dani-vaibhav 5a6705a
feat: add graphql-depth-limit dependency and re-enable depth limit va…
dani-vaibhav 4f8ac47
refactor: Resolving sorcery ai comments through : streamline action b…
dani-vaibhav 34bcafd
refactor: enhance type safety in aggregation results and mapping for …
dani-vaibhav f9692df
refactor: improve type safety and clarity in aggregation result mappi…
dani-vaibhav 5566bec
refactor: update mock server initialization to dynamically bind to an…
dani-vaibhav e247aad
refactor: enhance auto-reject logic for overlapping requests and impr…
dani-vaibhav 9f11967
refactor: update reservation request handling to use listing referenc…
dani-vaibhav 23ef69c
refactor: improve sharerId handling and simplify loading state manage…
dani-vaibhav 826b507
refactor: remove unnecessary helper abstractions and simplify contain…
dani-vaibhav e4e523d
refactor: inline action handlers in requests components
dani-vaibhav 5c94fd3
refactor: remove redundant error handling from requests container
dani-vaibhav 52ce35f
refactor: simplify auto-reject logic by checking request state directly
dani-vaibhav c0b68ac
refactor: simplify myListingsRequests query by removing pagination an…
dani-vaibhav e842b41
refactor: simplify repository mapping logic per PR feedback
dani-vaibhav a78b786
refactor: update UI to match simplified GraphQL schema
dani-vaibhav 5354d77
fix : Data population
dani-vaibhav 97fe325
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav e762bfc
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 68170d9
fix: remove unused useNavigate import
dani-vaibhav 9a702a8
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 5391e9a
PR comments fix
dani-vaibhav 35f019f
refactor: modularize action buttons and cell rendering in AllListings…
dani-vaibhav efb5d3d
fix: update reservation request states and properties for clarity
dani-vaibhav 361292f
fix: add pnpm overrides for qs package to resolve version conflicts
dani-vaibhav 0be20b3
refactor: reorganize imports and improve code formatting for readability
dani-vaibhav 003b0d7
fix: update environment variable access to use bracket notation for c…
dani-vaibhav b19d2e1
refactor: streamline action rendering logic in RequestsTable component
dani-vaibhav 238bf27
feat: implement AllListingsTable component with enhanced filtering an…
dani-vaibhav cc81763
fix: change STATUS_OPTIONS export to const for consistency
dani-vaibhav 59bbf18
fix: update environment variable access for admin and user profiles f…
dani-vaibhav 4d77a7e
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 1f4a66f
fix: correct pnpm configuration by consolidating overrides and auditC…
dani-vaibhav 558a52e
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 175e460
feat: add listing title and status filter components
dani-vaibhav 7996028
feat: refactor listings table components and remove unused files
dani-vaibhav 2fe1243
refactor: clean up requests table components for improved readability
dani-vaibhav 5683e61
refactor: clean up requests table components for improved readability
dani-vaibhav e792066
fix: correct pnpm configuration by consolidating overrides and auditC…
dani-vaibhav 7435bfd
Merge branch 'main' into copilot/sub-pr-257
dani-vaibhav 6c51ae6
ProfilePage updated
dani-vaibhav File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -18,3 +18,6 @@ __* | |
| apps/ui-sharethrift/tsconfig.tsbuildinfo | ||
| **/node_modules | ||
| **/dist | ||
|
|
||
| # Log files | ||
| *.log | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
13 changes: 6 additions & 7 deletions
13
...ayouts/home/components/view-listing/listing-information/listing-information.container.tsx
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
12 changes: 6 additions & 6 deletions
12
...mponents/layouts/home/components/view-listing/listing-information/listing-information.tsx
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
37 changes: 37 additions & 0 deletions
37
...sharethrift/src/components/layouts/home/my-listings/components/AllListingsTableHeader.tsx
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same idea here as the above comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,37 @@ | ||
|
|
||
| import { Input } from 'antd'; | ||
| const { Search } = Input; | ||
|
|
||
| interface ListingTitleFilterDropdownProps { | ||
| readonly onSearch: (value: string) => void; | ||
| readonly searchText: string; | ||
| readonly setSelectedKeys: (selectedKeys: React.Key[]) => void; | ||
| readonly selectedKeys: React.Key[]; | ||
| readonly confirm: () => void; | ||
| } | ||
|
|
||
| export function ListingTitleFilterDropdown({ | ||
| onSearch, | ||
| searchText, | ||
| setSelectedKeys, | ||
| selectedKeys, | ||
| confirm, | ||
| }: ListingTitleFilterDropdownProps): React.ReactNode { | ||
| return ( | ||
| <div style={{ padding: 8 }}> | ||
| <Search | ||
| placeholder="Search listings" | ||
| value={selectedKeys.length ? (selectedKeys[0] as string) : searchText} | ||
| onChange={(e) => { | ||
| setSelectedKeys(e.target.value ? [e.target.value] : []); | ||
| }} | ||
| onSearch={(value) => { | ||
| confirm(); | ||
| onSearch(value); | ||
| }} | ||
| style={{ width: 200 }} | ||
| allowClear | ||
| /> | ||
| </div> | ||
| ); | ||
| } |
11 changes: 11 additions & 0 deletions
11
...sharethrift/src/components/layouts/home/my-listings/components/AllListingsTableStatus.tsx
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| import { Tag } from 'antd'; | ||
|
|
||
| import { getStatusTagClass } from './status-tag-class.ts'; | ||
|
|
||
| export function AllListingsTableStatus({ | ||
| status, | ||
| }: Readonly<{ | ||
| status: string; | ||
| }>): React.ReactNode { | ||
| return <Tag className={getStatusTagClass(status)}>{status}</Tag>; | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.