Skip to content

(Online shop) seller administration, sale-items, order-review, order checkout & order status #380

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
wants to merge 44 commits into
base: dev
Choose a base branch
from

Conversation

adisa39
Copy link
Collaborator

@adisa39 adisa39 commented Apr 4, 2025

  • Added Reset button to seller order items.
  • Revamp sales items list to refresh picked items on successful order checkout.
  • Improve seller order, order-items fulfillment UI to show tides color based on status.
  • Integrate Reset, Fulfill & Refund buttons with BE API.
  • improve order checkout edge cases on cancel, on error and successful.
  • Revamp payment types to accommodate membership payment.
  • Include "Review Order" button on the side nav bar.
  • Added buyer-orders-reviews screen.
  • Added buyer-order-details screen.

The item checkout workflow is now complete as user can pick items from seller, checkout orders, make u2u payment and review ordered items, also Sellers can check buyers' orders, mark orders/items as fulfilled or refunded.

adisa39 and others added 24 commits January 14, 2025 17:54
@adisa39 adisa39 changed the title (Online shop) seller administration, sale-items, order Checkout & order status (Online shop) seller administration, sale-items, order-review, order checkout & order status Apr 18, 2025
@swoocn swoocn added the in progress WIP label May 28, 2025
@@ -221,57 +246,60 @@ export default function BuyFromSellerForm({ params }: { params: { id: string } }
</div>

{/* Online Shopping */}
{isOnlineShoppingEnabled && (
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reason why the toggle is removed..? 🤔

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! 👍

// Import Axios client and logger
import axiosClient from "@/config/client";
import logger from '../../logger.config.mjs';
import { PaymentDataType } from "@/constants/types";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This import is not being used.

import { PaymentDataType } from "@/constants/types";

// Create or Update an Order
export const createOrUpdateOrder = async (orderData: any) => {
Copy link
Member

@swoocn swoocn May 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a big fan of consolidating create and update logic into a single endpoint. It would be cleaner and more maintainable in the long run to distinguish these operations using separate POST and PUT endpoints. Definitely a refactoring opportunity for later.

@swoocn
Copy link
Member

swoocn commented Jun 5, 2025

Hi @adisa39 - great job overall!
I've more or less completed my review of your PRs for both the FE and BE. Please go ahead and perform regression testing in the sandbox environment to ensure everything is working as expected. The only item currently outstanding is the availability of a multisig wallet in the Testnet environment . I'm actively working with the PiCT to get that set up and will keep you posted accordingly. Thx!

@swoocn swoocn added pending-c/o-feedback Awaiting feedback from Code Owner. and removed in progress WIP labels Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending-c/o-feedback Awaiting feedback from Code Owner.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants