DEV Community

Cover image for How to contribute to DEV this hacktoberfest

How to contribute to DEV this hacktoberfest

Ben Halpern on October 01, 2019

### **First of all, if you haven't yet done so...** ### **[⭐️⭐️⭐️ Add a star to the dev.to repo on GitHub ⭐️⭐️⭐️](https://github.com/thepractical...
Read full post
Collapse
 
lightalloy profile image
Anna Buianova •

A couple of issues from me to contribute to:

Few items left to move delays to ActiveJob:

Move delay calls to ActiveJob #3136

lightalloy avatar
lightalloy commented on Jun 12, 2019

As described in #2497, to be less dependent of DelayedJob we also need to move jobs which are created by calling delay method to ActiveJob:

Here is the list of the calls:

  • [x] Slackbot.ping
  • [ ] MembershipMailer.delay.new_membership_subscription_email
  • [ ] MembershipMailer.delay.subscription_update_confirm_email
  • [ ] MembershipMailer.delay.subscription_cancellation_email
  • [ ] user.follow
  • [ ] reaction.create
  • [ ] ArticleAnalyticsFetcher.new.update_analytics
  • [ ] HtmlVariantSuccess.create
  • [ ] HtmlVariantTrial.create
  • [ ] message.send_push
  • [ ] RssReader.new.fetch_user
  • [ ] several calls in trigger_delayed_index

Actions for each of the calls:

  • create a corresponding ActiveJob, specify a queue name
  • call the required method inside a job
  • replace the delay method calls with a job perform_later call
  • you may need to modify the existing tests by using perform_enqueued_job instead of run_background_jobs_immediately helper

Remember to pass record ids (if needed) instead of ActiveRecord objects to a job to avoid deserialization errors (#1621)

</div>
<div class="gh-btn-container"><a class="gh-btn" href="https://github.com/thepracticaldev/dev.to/issues/3136">View on GitHub</a></div>


Remove unused methods:

Remove unused `_without_delay` methods #2950

lightalloy avatar
lightalloy commented on May 24, 2019

While asynchronous methods are being moved to separate Active Jobs (#1996, #2497), we keep _without_delay methods for a while, because the jobs that use this method can still exist at the moment of deploy. But eventually, we need to delete them to declutter because most of them are not actively used in the codebase. Some of these methods are used in the codebase explicitly. I suppose that it's worth to replace such calls with the perform_now job calls. But at least we should delete the unused methods.

</div>
<div class="gh-btn-container"><a class="gh-btn" href="https://github.com/thepracticaldev/dev.to/issues/2950">View on GitHub</a></div>


Display podcast episode's published_at date #3498

lightalloy avatar
lightalloy commented on Jul 17, 2019

I would like to see published_at date of an episode on its page, in the podcasts feed and on the podcast's episodes list. It would allow me to understand if the episode is fresh and if the podcast is active.

</div>
<div class="gh-btn-container"><a class="gh-btn" href="https://github.com/thepracticaldev/dev.to/issues/3498">View on GitHub</a></div>


Collapse
 
ben profile image
Ben Halpern •

I'm already on my way 😉

Collapse
 
jess profile image
Jess Lee •

I can't believe so many people have already completed the challenge 🙀

thepracticaldev image

Hacktoberfest Completion Thread

dev.to staff ・ Sep 30 ・ 1 min read

#hacktoberfest
Collapse
 
amorpheuz profile image
Yash Dave •

Hyped to continue contributing to the repo throughout hacktoberfest! Good luck to all the contributors.

Collapse
 
ben profile image
Ben Halpern •

Woohoo!

Collapse
 
terceranexus6 profile image
Paula •

Already sent a PR! I wanted to be able to add my mastodon instance for a while now... I would be happy if I can contribute with your repo as well!