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

fix: mindshare related articles #2340

Merged
merged 4 commits into from
Jan 30, 2024
Merged

Conversation

arzljames
Copy link
Contributor

@arzljames arzljames commented Jan 29, 2024

ISSUES:

  1. Not showing related article(s) based on content from the manager
    image
    image

  2. Current article is showing in the related articles section
    image
    Screenshot 2024-01-30 021422

FIXED:
Prioritized to render (if there's any) related articles from content. Otherwise render the latest articles excluding the current article page.

2024-01-30.02-22-03.mp4

@arzljames arzljames changed the title Fix: Mindshare related articles fix: mindshare related articles Jan 29, 2024
@arzljames arzljames mentioned this pull request Jan 29, 2024
@arzljames arzljames linked an issue Jan 29, 2024 that may be closed by this pull request
darwin808
darwin808 previously approved these changes Jan 30, 2024
@darwin808 darwin808 merged commit fe58a49 into stage Jan 30, 2024
5 checks passed
@darwin808 darwin808 deleted the fix/mindshare-related-articles branch January 30, 2024 00:20
@darwin808 darwin808 mentioned this pull request Jan 30, 2024
darwin808 pushed a commit that referenced this pull request Jan 30, 2024
**ISSUES:**

1. Not showing related article(s) based on content from the manager

![image](https://github.com/zesty-io/website/assets/70579069/f90784bd-6d34-48ce-bf5e-ffe902ae484b)

![image](https://github.com/zesty-io/website/assets/70579069/ee3b9ed5-953c-4c10-876d-65bf692b0f2b)

2. Current article is showing in the related articles section

![image](https://github.com/zesty-io/website/assets/70579069/e09505a3-9720-4f10-a849-ed461ab99392)
![Screenshot 2024-01-30
021422](https://github.com/zesty-io/website/assets/70579069/631df9e6-f151-4b04-8c71-621c3cb7eb59)


**FIXED:** 
Prioritized to render (if there's any) related articles from content.
Otherwise render the latest articles excluding the current article page.



https://github.com/zesty-io/website/assets/70579069/b1db6a37-e6f2-41aa-ba68-e7516bf93879

---------

Co-authored-by: darwin.apolinario <darwin.apolinario@gmail.com>
darwin808 pushed a commit that referenced this pull request Jan 30, 2024
**ISSUES:**

1. Not showing related article(s) based on content from the manager

![image](https://github.com/zesty-io/website/assets/70579069/f90784bd-6d34-48ce-bf5e-ffe902ae484b)

![image](https://github.com/zesty-io/website/assets/70579069/ee3b9ed5-953c-4c10-876d-65bf692b0f2b)

2. Current article is showing in the related articles section

![image](https://github.com/zesty-io/website/assets/70579069/e09505a3-9720-4f10-a849-ed461ab99392)
![Screenshot 2024-01-30
021422](https://github.com/zesty-io/website/assets/70579069/631df9e6-f151-4b04-8c71-621c3cb7eb59)


**FIXED:** 
Prioritized to render (if there's any) related articles from content.
Otherwise render the latest articles excluding the current article page.



https://github.com/zesty-io/website/assets/70579069/b1db6a37-e6f2-41aa-ba68-e7516bf93879

---------

Co-authored-by: darwin.apolinario <darwin.apolinario@gmail.com>
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.

🐛 [Bug]:
2 participants