search

malmostad / sitevision_assets

Sitevision specific assets used in addition to Global Assets
http://malmostad.github.io/wag-external-v4/
GNU Affero General Public License v3.0
0 stars 1 forks source link

Flytt av portlet-kod till Github #107

Closed olajoh1 closed 10 years ago

olajoh1 commented 10 years ago
  1. Hur hanterar du källkoden för de ändringar du gjort? Vi kan inte se att det finns något commitat sen 10 månader i Trac. Andra utvecklare här committar ett par gånger om dan när det arbetar med förändringar eller nyutveckling.
  2. Vi har flyttat över all annan källkod till publika repon på Github, bl.a. det som dina kolleger jobbar med och det är lämpligt att vi flyttar kontaktrutan/-formuläret dit med till ett eget repo. Hur lång tid skulle det ta dig att kolla igenom koden så att där inte ligger lösenord eller annat till databas eller i gamla deploymentscript och sen checka in koden på Github. Vi tar inte med historiken från Subversion utan startar med den senaste versionen du har som får bli första versionen på Github. Det behövs också en enkel README.md för hur man bygger, deployar etc. Titta på den här t.ex.:

https://github.com/malmostad/sitesearch

CC: @martent @susjon11

erikdybner commented 10 years ago
  1. Eftersom jag blev utlåst ur SVN under en längre period när jag fick ett nytt VPN-konto till Malmö så började jag spara mot en server på CGI istället, nu har jag dock tillgång igen och committat de senaste ändringarna till Malmös SVN.
  2. Det ser relativt bra ut vad gäller lösenord och dylikt i koden, sådant ligger i separata properties-filer. Däremot behöver man ändra byggscriptet som innehåller en del servernamn och lösenord. Man behöver kanske ca åtta timmar för att leta igenom kod efter eventuella lösenord eller annat samt att ändra byggscriptet. En README.md-fil i stil med exemplet kan vi också skriva, den blir kortfattad eftersom man bygger projektet genom att helt enkelt köra byggscriptet (för detta krävs JDK och Ant)

En övrig kommentar: Förutom koden för Kontaktruta Stadsområden och Skriv till oss innehåller kontaktruta-projektet även kod för portlets som inte används längre på webbplatsen så vitt jag vet: Stadsdelsinformation, Kontaktuppgifter, Kontaktruta(gamla versionen).

olajoh1 commented 10 years ago

Hej igen! Vi vill att du migrerar över din senaste version till Github efter att du har rensat. Fortsätt sedan att jobba mot Github. Skriv också en readme.

De data, t.ex.stadsdelsinfo, som inte används behöver inte migreras över.

@martent - skapa repo.

martent commented 10 years ago

@erikdybner Jag har satt upp repot https://github.com/malmostad/sitevision-contact-portlet och gett dig fulla rättigheter.

Skapa en git-tag för varje deploy i produktion med semantiskt versionsnamn och lägg till en kort release-not i Github typ: https://github.com/malmostad/intranet-dashboard/releases

martent commented 10 years ago

@erikdybner Glöm inte att commita dagligen eller oftare till SVN fram tills du migrerat till Github. En disk-krasch på din laptop kommer att stoppa releasen annars.

erikdybner commented 10 years ago

Hej, projektet Kontaktruta i SVN innehåller ett antal portlets: Kontaktruta (gamla kontaktrutan) Kontaktruta Stadsområden (nuvarande kontaktrutan) Kontaktinfo Skriv till oss Stadsdelsinformation (gammal portlet, nyheter per stadsdel) Kod för att anropa webservice för PrimeCase ärendehanteringssystem

Ska jag rensa samtligt förutom nuvarande kontaktrutan? Jag tror inte att någon av de andra portletarna används nu längre (rätta mig om jag har fel). Ska jag dessutom rensa bort all kod som har med PrimeCase att göra?

martent commented 10 years ago

Ju renare desto bättre. Det är nuläget vi startar med på Github.

Gör så att du checkar in det som ska lanseras nu, SVN får fungera arkiv. Ta gärna bort PrimeCase och annat som inte används. Och givetvis känslig data.

erikdybner commented 10 years ago

Nu har jag lagt över koden till GitHub, det är en rensad version av projektet som inte innnehåller all den legacy-kod som körs i produktion idag.

martent commented 10 years ago

Utmärkt!

Kan du ta bort .svn-mapparna och commita?

Görs det fortfarande anrop till WSDL:n esb.handlecase.url eller den är utfasad? Det var den stubbade lösningen för PrimeCase.

legander commented 10 years ago

Ta bort .svn-mapparna från repot. @rbolton479

martent commented 10 years ago

Du måste acceptera din invite @rbolton479 som medlem i teamet CGI. Du ska ha fått ett email från GitHub.

rbolton479 commented 10 years ago

Kommer börja med allt imorgon torsdag, min CGI mail konto blev avstängde så jag har inte hunnit läsa mejl i två dagar.

rbolton479 commented 10 years ago

Jag har tagit bort .svn mapparna nu och försöka 'push' till github men det går inte. Jag antar att det har någonting med invite till CGI gruppen som är problemet. Jag hittar ingen mejl angående invite till gruppen.

martent commented 10 years ago

Du kan accepta din invite genom att gå till https://github.com/malmostad

martent commented 10 years ago

Glöm inte att sätta din e-mail i dina git-settings på din dator, @hryd vet hur du gör.

rbolton479 commented 10 years ago

googled it, email address, fixed.

rbolton479 commented 10 years ago

nu är svn mapparna borta

martent commented 10 years ago

Bra. Se mail om ny repo-adress.

rbolton479 commented 10 years ago

project cloned with new repo-address :-)