On Wrapping it up with Convert Medium articles from Medium to native site#86
Open
john-walter-munene wants to merge 2 commits intoStabilityNexus:mainfrom
Open
On Wrapping it up with Convert Medium articles from Medium to native site#86john-walter-munene wants to merge 2 commits intoStabilityNexus:mainfrom
john-walter-munene wants to merge 2 commits intoStabilityNexus:mainfrom
Conversation
Contributor
📝 WalkthroughWalkthroughThis PR standardizes article media/markup (converting Markdown images to semantic figure/figcaption, normalizing /images/ paths, adjusting emphasis syntax, and removing some img alt attributes), adds/updates article content (small text augmentations), and bulk-appends many entries to the article index JSON. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 7
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
public/articles/articles-index.json (1)
13-18:⚠️ Potential issue | 🟡 MinorEmpty title will render a blank listing.
Line 14 sets
titleto an empty string; ensure the UI has a fallback or restore the title to avoid empty cards.
🤖 Fix all issues with AI agents
In `@public/articles/addressing-the-stablecoin-trilemma.md`:
- Around line 36-39: Add descriptive alt attributes to each <img> inside the
<figure> elements (e.g., the image showing "stablecoins as the backbone of our
digital economy") so screen readers and accessibility tools have meaningful
text; update every <img> in the file where similar figures appear (the ones
paired with their <figcaption> lines) to include an appropriate alt="..." that
succinctly describes the image content.
- Around line 58-61: The img src attributes inside the <figure> blocks contain
double slashes (e.g. <img
src="/images//Crypto-Collateralized%20Stablecoins.webp">) which can break some
hosts; find each <img src="/images//..."> occurrence in the article (the
<figure> elements and their <img> tags) and normalize the path to a single slash
(replace "/images//" with "/images/") for all instances on the page.
In `@public/articles/articles-index.json`:
- Around line 156-164: The metadata for the article with slug
"retrospective-2023" has the author and date fields swapped/incorrect (author is
set to a date string and date is set to 2024-06-13); open the JSON entry for
slug "retrospective-2023" and correct the values so "author" contains the
author's name and "date" contains the intended publication date (or swap them if
they were accidentally reversed), ensuring the date is in the same format as
other entries and the author is a string name.
In `@public/articles/buy-and-hodl-sigusd-rsv-on-cardano.md`:
- Around line 47-49: The <figure> block is missing accessible content: add an
appropriate alt attribute to the <img> tag (e.g., alt="Rosen bridge
illustration" or other descriptive text matching the image) and include a
<figcaption> element inside the <figure> that provides a short caption
describing the image/context; update the <img src="/images/rosen-bridge.webp">
to include the alt text and add a <figcaption> under it so the figure is both
accessible and semantically complete.
In `@public/articles/understanding-why-and-how-stablecoins-depeg.md`:
- Line 48: The five <img> tags in this markdown (example tag: <img
src="/images/USDT depegging to $0.95 shortly after the LUNA UST crisis. Source
CoinMarketCap.webp">) are missing alt attributes; add concise, descriptive alt
text to each <img> reflecting the image content/purpose (e.g., "USDT price
depegging to $0.95 after LUNA/UST crisis"), or if an image is purely decorative
use alt="" to mark it as decorative; ensure each img tag in this file includes
an appropriate alt value.
In `@public/articles/unleashing-the-potential-of-djed-powered-by-coti.md`:
- Line 55: Several <img> tags in the markdown lack alt attributes (e.g., the tag
with src "/images/COTI falsely claiming that Djed powered by COTI is fully
decentralized.webp"); add concise, descriptive alt text to each of the seven
missing <img> elements so screen readers can convey the image meaning (or use
alt="" for purely decorative images), and ensure the same descriptive style as
the existing images in this file for consistency across the document.
In `@public/articles/what-is-djed-stablecoin-protocol.md`:
- Around line 49-52: The <img> inside the <figure> (the element containing "How
Djed Works") is missing an alt attribute; add a meaningful alt text to the <img>
tag (e.g., alt="Diagram showing how Djed stablecoin works, including reserve and
mint/burn flow") or an empty alt (alt="") if the image is purely decorative, so
screen readers get proper context; update the <img
src="/images/how-djed-works.webp"> element accordingly.
🧹 Nitpick comments (3)
public/articles/djed-stablecoin-oracles-across-chains.md (2)
29-32: Avoid URL-encoded spaces in image paths.The image path contains a URL-encoded space (
%20). Use hyphens or underscores instead for better compatibility and readability.♻️ Proposed fix
Rename the image file from
Overview%20on%20Djed-based%20Stablecoins.webptooverview-on-djed-based-stablecoins.webpand update the reference:<figure> - <img src="/images/Overview%20on%20Djed-based%20Stablecoins.webp"> + <img src="/images/overview-on-djed-based-stablecoins.webp" alt="Overview on Djed-based Stablecoins"> <figcaption>Overview on Djed-based Stablecoins</figcaption> </figure>
30-30: Add alt attribute for accessibility.The
<img>tag is missing thealtattribute. While the<figcaption>provides context, screen readers benefit from explicit alt text.♿ Proposed fix
- <img src="/images/Overview%20on%20Djed-based%20Stablecoins.webp"> + <img src="/images/Overview%20on%20Djed-based%20Stablecoins.webp" alt="Overview on Djed-based Stablecoins">public/articles/the-history-of-djed-and-the-djed-alliance.md (1)
36-36: Add alt attribute for accessibility.The
<img>tag is missing thealtattribute. While the<figcaption>provides context, explicit alt text improves accessibility for screen readers.♿ Proposed fix
- <img src="/images/Djed and Djed Alliance Full History.webp"> + <img src="/images/Djed and Djed Alliance Full History.webp" alt="Timeline showing the full history of Djed and Djed Alliance"> <figcaption>Full History of Djed Alliance</figcaption>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This commit:
![]()to use<figure></figure>and thus now shows up website__to use**thus all italicised text shows up well as pre-processor for markdown syntax didn't recognize__This commit wraps up the conversion of articles from Medium to the native site.
It also recommends the update of all articles on Medium to use canonical urls that now point to this website.
Summary by CodeRabbit
New Features
Documentation