Skip to content

(Online shop) seller registration, order fulfilment #368

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 19 commits into
base: dev
Choose a base branch
from

Conversation

adisa39
Copy link
Collaborator

@adisa39 adisa39 commented Feb 26, 2025

  • Renamed Seller Registration screen to Seller Administration.
  • Added Pi payment configuration.
  • Created Seller Order Fulfillment screen.
  • Updated Seller Administration Screen to include Online-Shop Order fulfillment section.
  • Updated Sale-Items screen to include Online-Shop Items section.
  • Included items checkout features.
  • Integration of Order Fulfillment API endpoint with BE.

@swoocn
Copy link
Member

swoocn commented Mar 2, 2025

Build is failing due to the same reason as your initial PR ->
Can you please address when you can @adisa39?

image

const sellerName = searchParams.seller_name;
const sellerType = searchParams.seller_type;

const translatedStockLevelOptions = [
Copy link
Member

Choose a reason for hiding this comment

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

I don't believe this collection is being used in this file.

@swoocn
Copy link
Member

swoocn commented Apr 1, 2025

Hi @adisa39 - Please remember to pull in the latest changes before making updates to help avoid merge conflicts, as they can be quite a hassle to resolve. It only takes two simple commands git fetch and then git pull. You should also run a git merge origin/dev to make sure you are synced with the dev branch on a daily basis.
Appreciate your help with this! 👌

),
},
];
{
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for keeping the alignment consistent! 👍

@@ -4,23 +4,24 @@ import { useTranslations, useLocale } from 'next-intl';
import Image from 'next/image';
import Link from 'next/link';

import React, { useEffect, useState, useContext, useRef } from 'react';
import React, { useEffect, useState, useContext, useRef, ChangeEvent } from 'react';
Copy link
Member

Choose a reason for hiding this comment

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

I don't believe ChangeEvent import is being used.


import ConfirmDialog from '@/components/shared/confirm';
import { Button, OutlineBtn } from '@/components/shared/Forms/Buttons/Buttons';
import TrustMeter from '@/components/shared/Review/TrustMeter';
import ToggleCollapse from '@/components/shared/Seller/ToggleCollapse';
import Skeleton from '@/components/skeleton/skeleton';
import { ISeller, IUserSettings, IUser, SellerItem } from '@/constants/types';
import { ISeller, IUserSettings, IUser, SellerItem, PaymentDataType, PickedItems, FulfillmentType } 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.

I don't believe PickedItems import is being used.

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