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.

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

Added Trie Category Questions #9

Merged
merged 3 commits into from
May 26, 2020
Merged

Conversation

mm1705
Copy link
Contributor

@mm1705 mm1705 commented May 25, 2020

@seanprashad I added some questions related to Trie category.

Since I am not a LC premium subscriber, these are the questions I have not solved with Trie tag:

Ref: #7.

Copy link
Owner

@seanprashad seanprashad left a comment

Choose a reason for hiding this comment

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

Thank you so much for your work here @mm1705! I have retrieved the companies for you to add as well as found another question that we should change the pattern from DFS to Trie.

Once those are fixed up, we should be able to land this PR and have it show up for everyone to see 🎉

src/data/index.js Outdated Show resolved Hide resolved
src/data/index.js Outdated Show resolved Hide resolved
src/data/index.js Outdated Show resolved Hide resolved
src/data/index.js Outdated Show resolved Hide resolved
src/data/index.js Outdated Show resolved Hide resolved
src/data/index.js Outdated Show resolved Hide resolved
src/data/index.js Outdated Show resolved Hide resolved
@seanprashad
Copy link
Owner

Also here are the 3 premium questions that you mentioned in the original post:

  {
    id: 169,
    name: 'Index Pairs of a String',
    url: 'https://leetcode.com/problems/index-pairs-of-a-string/',
    pattern: 'Trie',
    difficulty: 'Easy',
    companies: ['Amazon'],
  },
  {
    id: 170,
    name: 'Design Search Autocomplete System',
    url: 'https://leetcode.com/problems/design-search-autocomplete-system/',
    pattern: 'Trie',
    difficulty: 'Hard',
    companies: ['Amazon', 'Google', 'Microsoft', 'Uber', 'Lyft'],
  },
  {
    id: 171,
    name: 'Word Squares',
    url: 'https://leetcode.com/problems/word-squares/',
    pattern: 'Trie',
    difficulty: 'Hard',
    companies: ['Oracle'],
  },

I think if you slot these into the right spots, you'll need to update the id accordingly (especially for Index Pairs of a String since it'll come before all Medium/Hard Trie questions)

@mm1705 mm1705 requested a review from seanprashad May 26, 2020 06:33
@mm1705
Copy link
Contributor Author

mm1705 commented May 26, 2020

Thank you for info from LC premium @seanprashad!

I have updated the file for review, let me know if any other changes are required. 🙂

Copy link
Owner

@seanprashad seanprashad left a comment

Choose a reason for hiding this comment

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

Great work @mm1705! I'm sure that everyone who comes across this list will definitely appreciate the work that you've put in 😁

I'm going to merge this patch into master but just a heads up that I'll need to fix some formatting errors for src/data/index.js - this project uses Prettier and Pretty-Quick to automatically format any changed files (.js and .json in this case) when you save and/or commit - it's super convenient!

@seanprashad
Copy link
Owner

Here's what the website will look like once published via npm run deploy:

image

@seanprashad seanprashad merged commit 56a3d43 into seanprashad:master May 26, 2020
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