Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Pop-up creations #198

Merged
merged 41 commits into from
Oct 20, 2024
Merged

Pop-up creations #198

merged 41 commits into from
Oct 20, 2024

Conversation

jamshidbek-choriyev
Copy link
Contributor

@jamshidbek-choriyev jamshidbek-choriyev commented Oct 7, 2024 •
edited
Loading

image
image
image
image
image
image
image

FirdavsAX
FirdavsAX reviewed Oct 7, 2024
View reviewed changes
Copy link

@FirdavsAX FirdavsAX left a comment

Choose a reason for hiding this comment

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

Your changes are very big . Please create smaller pull requests for easey reviewing

Ramadan1011
Ramadan1011 requested changes Oct 7, 2024
View reviewed changes
Copy link
Contributor

@Ramadan1011 Ramadan1011 left a comment

Choose a reason for hiding this comment

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

if you don't use commented lines, it's better to remove it

@jamshidbek-choriyev
Update Create.cshtml
2c644d4 
remove commented
@jamshidbek-choriyev
Update Index.cshtml
d58893a 
removid commeted
@jamshidbek-choriyev
Update CreateByButton.cshtml
c18b7ba 
 deleted the comments
@jamshidbek-choriyev
Update Create.cshtml
80eec22 
 deleted the comments
@jamshidbek-choriyev
Update Create.cshtml
c23b4b9 
 deleted the comments
@Ramadan1011 Ramadan1011 self-requested a review October 7, 2024 17:49
Ramadan1011
Ramadan1011 approved these changes Oct 7, 2024
View reviewed changes
Copy link
Contributor

@Ramadan1011 Ramadan1011 left a comment

Choose a reason for hiding this comment

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

looks good

Ramadan1011
Ramadan1011 approved these changes Oct 8, 2024
View reviewed changes
Copy link
Contributor

@Ramadan1011 Ramadan1011 left a comment

Choose a reason for hiding this comment

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

looks good!

@jamshidbek-choriyev jamshidbek-choriyev self-assigned this Oct 15, 2024
@Mirazyzz Mirazyzz merged commit f9d1e15 into DiyorMarket:integration Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@diyorzz diyorzz diyorzz approved these changes

@Boburm1rzo Boburm1rzo Boburm1rzo approved these changes

@Ramadan1011 Ramadan1011 Ramadan1011 approved these changes

@Mirazyzz Mirazyzz Awaiting requested review from Mirazyzz

@FirdavsAX FirdavsAX Awaiting requested review from FirdavsAX

@SharifovDeveloper SharifovDeveloper Awaiting requested review from SharifovDeveloper

@khojianvar khojianvar Awaiting requested review from khojianvar

@AzamatG17 AzamatG17 Awaiting requested review from AzamatG17

@aXayitov aXayitov Awaiting requested review from aXayitov

Assignees

@jamshidbek-choriyev jamshidbek-choriyev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@jamshidbek-choriyev @diyorzz @Boburm1rzo @FirdavsAX @Ramadan1011 @Mirazyzz