Skip to content
Snippets Groups Projects

add text about refactoring to devdocs

Merged Caluera requested to merge calu-refactor-devdocs into master
All threads resolved!

Closes None

What does this MR do?

Adds a text about refactoring to the devdocs

How confident are you it won't break things if deployed?

It is just a devdocs article, please look over if it is correct :)

Links to related issues

Was discussed during https://yunity.slack.com/archives/CD7HJ1HHC/p1587823492001900 https://codi.kanthaus.online/2020-04-25_code_refactoring?both#

How to test

See https://devdocs.foodsharing.network/about-devdocs.html#setting-things-up

Screenshots (if applicable)

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Caluera

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Alex approved this merge request

    approved this merge request

  • Flukx
  • Flukx
  • Flukx
  • Caluera added 2 commits

    added 2 commits

    • 39bdaafc - make links in refactor article pretty
    • 2a7c778e - move basic explanations to php.md

    Compare with previous version

  • Caluera added 48 commits

    added 48 commits

    Compare with previous version

  • Flukx mentioned in merge request !1463 (merged)

    mentioned in merge request !1463 (merged)

  • Jano resolved all threads

    resolved all threads

  • Jano approved this merge request

    approved this merge request

  • Jano added 1 deleted label

    added 1 deleted label

  • merged

  • No beta testing needed for this one

  • Chris Oelmueller removed 1 deleted label

    removed 1 deleted label

  • Christian Walgenbach mentioned in merge request !1509 (merged)

    mentioned in merge request !1509 (merged)

  • Please register or sign in to reply
    Loading