Ben Halpern Community 🌱

Ben Halpern 🦁
Ben Halpern 🦁

Posted on

Test!

dsdsdsewew

Top comments (1)

Collapse
 
ben profile image
Ben Halpern 🦁 • Edited

Quite a productive week!


I got my 8th Warm Welcome badge. I did mention I will stop at 10, but I want to see how far I can go. Also got 25 likes on the Meme Monday. Great to have these :)

Image


Got a lot of engagement on my guide to Dev.to! Though I didn't receive a badge for the #howtodevto yet. Either way, great achievement :D

Get Started on Dev.to! A Beginner's Guide to Engage with the Community! 💡 - DEV Community

All Images are own by the Pokemon Company! Important Information: This post is an Unofficial Dev.to...

favicon dev.to

Got 2 PR's merged into Forem this week!

  • The Quickie Post is fixed because there were cases where the characters were counted incorrectly and now the post only counts characters and not white spaces.

Ignore White Spaces when publishing a Quickie Post #23015

What type of PR is this? (check all applicable)

  • [ ] Refactor
  • [ ] Feature
  • [x] Bug Fix
  • [ ] Optimization
  • [ ] Documentation Update

Description

This PR addresses the issue where the condition is not checked properly when publishing a Quick Post. Added an additional thing in the condition where it removes the white spaces and only should count the characters and not the spaces.

elsif title.gsub(/\p{Space}+/u, '').length > max_length

Related Tickets & Documents

  • Related Issue #23009
  • Closes #23009

QA Instructions, Screenshots, Recordings

Make sure you are signed in

  • On the main page, click on "What's on your mind?"/ Quickie Post Section
  • Paste this text given by the issuer:
While setting up a Next.js project today, 

I noticed AGENTS & CLAUDE markdown files pointing to node_modules/next/dist/docs/.

Fascinating how the ecosystem evolving from monolithic to micro frontends.
and toward more integrated developer experiences.
  • Click on Post

What gif best describes this PR or how it makes you feel?

Happy

  • The "Report Abuse" button, located on the left side where the three dots are, is invisible if you were to look at your own post.

Hide "Report Abuse" button when User viewing their own Article #22958

What type of PR is this? (check all applicable)

  • [ ] Refactor
  • [x] Feature
  • [x] Bug Fix
  • [ ] Optimization
  • [ ] Documentation Update

Description

This PR address where if the User is viewing their own article, the option to "Report Abuse" will not show to the user.

image

However, if the user looks at other people's post (that the user do not own), it will show the option:

image

Note that if they are not sign in, the "Report Abuse" button will be visible by default.

This will prevent unnecessary self-report in my opinion because I believe there will be cases where a user will self-report himself either accidentally or intentionally. This will make it harder for the user to self-report.

Additionally, to the issue #14658, it was requested that the footer should remain:

image

Related Tickets & Documents

  • Related Issue #14658
  • Closes #14658

QA Instructions, Screenshots, Recordings

When you are signed in:

  1. View your own post
  2. Click the "..." button on the side.
  3. You will see that the "Report Abuse" button is not visible.
  4. Now, view any article and click the "..." and you will see the option.

When you are not signed in:

  • View any article, click the "..." dropdown and you will see the "Report Abuse" button.

Added/updated tests?

  • [ ] Yes
  • [ ] No, and this is why: please replace this line with details on why tests have not been included
  • [x] I need help with writing tests

What gif best describes this PR or how it makes you feel?

Pikachu


Awesome week I had! Can't wait for next week :D
Image