Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(app): add questions pagination #403

Merged
merged 10 commits into from
Dec 12, 2022
Merged

Conversation

AdiPol1359
Copy link
Contributor

Fixes #357

@vercel
Copy link

vercel bot commented Dec 12, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
devfaq ⬜️ Ignored (Inspect) Dec 12, 2022 at 9:37AM (UTC)

@github-actions
Copy link

github-actions bot commented Dec 12, 2022

📦 Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action 🤖

⚠️ Global Bundle Size Increased

Page Size (compressed)
global 79.03 KB (🟡 +1 B)
Details

The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!

@AdiPol1359 AdiPol1359 requested a review from typeofweb December 12, 2022 08:49
}) {
const page = parseInt(params.page);

if (!technologies.includes(params.technology) || isNaN(page)) {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
if (!technologies.includes(params.technology) || isNaN(page)) {
if (!technologies.includes(params.technology) || Number.isNaN(page)) {

@typeofweb typeofweb merged commit c20b11a into develop Dec 12, 2022
@typeofweb typeofweb deleted the 357-questions-pagination branch December 12, 2022 09:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Paginacja
2 participants