fix: domains with - were not loading the keywords.

resolves: #11
towfiqi · Dec 2, 2022 · efb565b · efb565b
1 parent a11b0f2
commit efb565b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pages/api/domains.ts b/pages/api/domains.ts
@@ -61,8 +61,8 @@ export const addDomain = async (req: NextApiRequest, res: NextApiResponse<Domain
    }
    const { domain } = req.body || {};
    const domainData = {
-      domain,
-      slug: domain.replaceAll('.', '-'),
+      domain: domain.trim(),
+      slug: domain.trim().replaceAll('-', '_').replaceAll('.', '-'),
       lastUpdated: new Date().toJSON(),
       added: new Date().toJSON(),
    };

diff --git a/pages/api/keywords.ts b/pages/api/keywords.ts
@@ -44,7 +44,7 @@ const getKeywords = async (req: NextApiRequest, res: NextApiResponse<KeywordsGet
    if (!req.query.domain && typeof req.query.domain !== 'string') {
       return res.status(400).json({ error: 'Domain is Required!' });
    }
-   const domain = (req.query.domain as string).replaceAll('-', '.');
+   const domain = (req.query.domain as string).replaceAll('-', '.').replaceAll('_', '-');
 
    try {
       const allKeywords:Keyword[] = await Keyword.findAll({ where: { domain } });