oxiton-foundation / click-metrics

Click Matrics is a powerful URL shortening and analytics platform designed to streamline the process of generating short URLs, creating QR codes, and tracking detailed analytics for each link.

https://click-metrics.vercel.app

MIT License

33 stars 49 forks source link

OOSI: Design the Link-in-bio page for click matrics #132

Closed lazybug19 closed 3 weeks ago

lazybug19 commented 4 weeks ago

Fixes #5

Screenshots

github-actions[bot] commented 4 weeks ago

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our #24. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

lazybug19 commented 3 weeks ago

@Swarnendu0123

Swarnendu0123 commented 3 weeks ago

I have already reviewed, please do the modifications mentioned in this https://github.com/oxiton-foundation/click-metrics/pull/132#pullrequestreview-2094642316

lazybug19 commented 3 weeks ago

No review comments are visible to me. The link says 0 reviews in total reviews section.

Swarnendu0123 commented 3 weeks ago

Please attach what the GitHub is showing.

lazybug19 commented 3 weeks ago

Swarnendu0123 commented 3 weeks ago

Sorry to say, but you should learn how to use GitHub first.

I can clearly see, I have reviewed these two comments which can be accessible in https://github.com/oxiton-foundation/click-metrics/pull/132#pullrequestreview-2094642316.

lazybug19 commented 3 weeks ago

hey, thank you for your kind feedback. Meanwhile check this github documentation Reviewing a pr since the ss you attached clearly shows your changes as pending and the documentation also states :

For further proof of no review requests on my side:

Swarnendu0123 commented 3 weeks ago

Let's give it a final touch, resolve the conflicts and we are done.

Swarnendu0123 commented 3 weeks ago

Thanks!

lazybug19 commented 3 weeks ago

@Swarnendu0123 The merged pr resolved issue #5 which had level 4 label when assigned and even now. Do rectify the level for merged pr accordingly.