search

podkrepi-bg / api

Nest.js REST backend for charity platform Podkrepi.bg https://podkrepi.bg/swagger
https://podkrepi.bg
MIT License
82 stars 43 forks source link

add findAll(GET) method in the campaign-application.service #651

Closed Martbul closed 2 months ago

Martbul commented 2 months ago

Closes #{issue number}

Motivation and context

Testing

Steps to test

New endpoints

Environment

New environment variables:

New or updated dependencies:

Dependency name Previous version Updated version Details
dependency/name v1.0.0 v2.0.0
github-actions[bot] commented 2 months ago

✅ Tests will run for this PR. Once they succeed it can be merged.

Martbul commented 2 months ago

Не знам защо ми е добавило старите комити от миналия ПР(този който беше мърджнат)

Martbul commented 2 months ago

PS: трябва да оправя тестовете(написани са за модела Campaign, вместо за CampaignApplication)

Martbul commented 2 months ago

PS2: В сийда на ДБ-то няма папка за Campaign-application

Martbul commented 2 months ago

Оправих тестовете!

sashko9807 commented 2 months ago

Не знам защо ми е добавило старите комити от миналия ПР(този който беше мърджнат) Защото има разлика от HEAD на локалния ти master бранч, с HEAD master branch на api-то. Това не е проблем, както виждаш в секцията Files Changes, са отразени само промените, които си ги намислил към този PR.

Ако не искаш да става така, локалния ти master branch трябва да е еднакъв с нашия master branch, т.е. локалния ти master branch трябва да си остане чист. За целта промените може да ги правиш в отделен branch.

Нов branch може да създадеш с командата git checkout -b NEW_BRANCH_NAME, като NEW_BRANCH_NAME ти е име с което желаеш да кръстиш бранча. Примерно за този PR може да е нещо от сорта: add-prisma-to-campaign-application. Командата автоматично ще те превключи към новия бранч.

След което commit-ваш промените и ги пушваш към този бранч. Ако до момента промените ги пушваш с командата git push origin master, вече ще ги пушваш с командата git push origin NEW_BRANCH_NAME. След което пускаш промените като PR от NEW_BRANCH_NAME.

В момента локалния ти master branch е "замърсен", т.е. има разминаване в HEAD както казах по рано. За да го оправиш може да направиш следното:

Взимаш commit hash на последния commit от нашия бранч преди твоите промени. Commit hash-a, можеш да го вземеш през github или чрез командата git log(за да напуспеш git log натисни q)

След като вземеш commit hash-a изпълняваш следната команда: git reset commitHash --hard, в случая предполагам за теб командата ще е git reset 7dc9b9b0a75f0999f618d336f2abd3926b9122c8 --hard, като с това ще се discard-нат всички локални промени в мастър branch.

Сега идва и момента да вземем последните промени от api repository-то. За да направим това първо трябва да се добави връзка към това repository. Това става с командата git remote add <name> <link> като: <name> ти е името с което ще реферираш Repository-то през git командата. <link> ти е линк на repository-то.

Примерно за теб командата може да е: git remote add podkrepi-api https://github.com/podkrepi-bg/api.git, За да видиш дали е успешно добавено можеш да стартираш командата git remote -v би трябвало да видиш podkrepi-api два пъти с (fetch) и (push)

След което за да издърпаш последните промени от api repository-то, изпълняваш следната команда git pull podkrepi-api master. Като podkrepi-api ти е връзката към repository-то, а master ти е branch-a от който ще pull-ваш промените.

И така ако всичко мине на ред master branch-a трябва да ти е еднакъв с нашия. Преди да започнеш работа по нов PR, дърпаш последните промени от този master branch, след което изпълняваш git checkout -b new-branch, и си правиш нужните промени по кода.