sam-myers / omnirss

https://omnirss.com
Apache License 2.0
0 stars 0 forks source link

Sweep: fix button on search page #125

Closed sam-myers closed 1 year ago

sam-myers commented 1 year ago

In functions/search/template.html, the link on line 15, <div class="container"><a class="navbar-brand" href="/public">OmniRSS</a><button data-bs-toggle="collapse" class="navbar-toggler" data-bs-target="#navcol-1"></button> is broken. It should lead to the main page at /, but it instead leads to /public.

Make these changes:

Checklist - [X] ``functions/search/template.html`` ✅ Commit c269fd8 - [X] ``cypress/e2e/search_results.cy.js`` ✅ Commit a3c8ca8
sweep-ai[bot] commented 1 year ago

Here's the PR! https://github.com/sam-myers/omnirss/pull/126.

⚡ Sweep Free Trial: I'm creating this ticket using GPT-4. You have 3 GPT-4 tickets left for the month and 1 for the day. For more GPT-4 tickets, visit our payment portal.

Actions (click)


Step 1: 🔎 Searching

I found the following snippets in your repository. I will now analyze these snippets and come up with a plan.

Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description. https://github.com/sam-myers/omnirss/blob/7ab53fd1df3fcb2710e62221a59efbe7f2193681/functions/search/template.html#L1-L51 https://github.com/sam-myers/omnirss/blob/7ab53fd1df3fcb2710e62221a59efbe7f2193681/cypress/e2e/search_results.cy.js#L1-L37

Step 2: ⌨️ Coding


Step 3: 🔁 Code Review

I have finished reviewing the code for completeness. I did not find errors for sweep/fix-button-navigation.

.


🎉 Latest improvements to Sweep:


💡 To recreate the pull request edit the issue title or description. To tweak the pull request, leave a comment on the pull request. Join Our Discord