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

Improve articles: summary, tags and some code syntax highlight #40

Merged
merged 4 commits into from
Mar 29, 2024

Conversation

ctmbl
Copy link
Contributor

@ctmbl ctmbl commented Mar 27, 2024

Few updates that comes from the desire of fixing Turtyo's post summary:
Screenshot_20240327_131926

  • fix turtyo's summary
  • homogenize tags
  • imrpove xlitoni's summary
  • fix turtyo's code blocks syntax highlight

@ctmbl ctmbl requested review from Turtyo, Xlitoni and ZynoXelek March 27, 2024 12:21
@Turtyo
Copy link

Turtyo commented Mar 27, 2024

Why is the text under my WU "Simple tutorial to discord bots using discord.py" ? I think it got mixed with the second blog post

Copy link

@Turtyo Turtyo left a comment

Choose a reason for hiding this comment

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

I don't have much to say about the changes, seems good to me

src/content/posts/web3py-solidity-write-up.md Show resolved Hide resolved
@ctmbl
Copy link
Contributor Author

ctmbl commented Mar 28, 2024

Why is the text under my WU "Simple tutorial to discord bots using discord.py" ? I think it got mixed with the second blog post

That was exactly my point see the first sentence of the PR comment 😉

Tks for the approval btw!

@ctmbl ctmbl requested a review from Enotias March 28, 2024 08:10
@Turtyo
Copy link

Turtyo commented Mar 28, 2024

That was exactly my point see the first sentence of the PR comment 😉

Oh that was the "before" not the "after", i see

@ctmbl ctmbl merged commit e8335b3 into iScsc:main Mar 29, 2024
4 checks passed
@ctmbl ctmbl deleted the improve-articles branch March 29, 2024 21:39
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.

3 participants