Skip to content

London_10 | Bedrije_Omuri |Module_DataBases _E-commerce| WEEK 3 - #109

Open
Bedi06 wants to merge 1 commit into
CodeYourFuture:mainfrom
Bedi06:main
Open

London_10 | Bedrije_Omuri |Module_DataBases _E-commerce| WEEK 3#109
Bedi06 wants to merge 1 commit into
CodeYourFuture:mainfrom
Bedi06:main

Conversation

@Bedi06

@Bedi06 Bedi06 commented Sep 9, 2023

Copy link
Copy Markdown

Learners, PR Template

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the requirements of this task

Changelist

Briefly explain your PR.

Questions

Ask any questions you have for your reviewer.

@sonarqubecloud

sonarqubecloud Bot commented Sep 9, 2023

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@sherif98 sherif98 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks great! Thanks for your work.

Comment thread E-Commerce/readme.md
- [ ] List the 5 most expensive products
- [ ] List all the products sold by suppliers based in the United Kingdom. The result should only contain the columns product_name and supplier_name
- [ ] List all the products whose name contains the word "socks":
select product_name from products where product_name like '%socks%';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please use uppercase letter for the SQL commands similar to the other queries.

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.

2 participants