search

systemapic / systemapic.js

Systemapic Javascript API
https://systemapic.com
0 stars 3 forks source link

Globesar slider2 #189

Closed jorgenevil closed 8 years ago

jorgenevil commented 8 years ago

This is the files with the sliders etc. They need to be called up in Gruntfile + app.ejs as well.

knutole commented 8 years ago

Cool. får du fiksa merge conflicts også? og lagd en PR på wu med grunt/app.ejs etc. ?

knutole commented 8 years ago

Og kanskje lagt den gigantiske arrayen i en egen fil?

knutole commented 8 years ago

Så er det sånn at både slider og chart-graph er i bigslider fila?

jorgenevil commented 8 years ago

Ja, slider og graph er i samme fila. Når det gjelder det å ha den psycho arrayen i separat fil, er det jo litt klønete siden de eksterne filene må kalles opp i grunt og ejs fila. Hva tenker du om det? Ha en data.js fil, som ikke synkes mellom portaler?

knutole commented 8 years ago

Vi kan legge den fila på server-side og bare requeste den (på samme måte som vi requester config, etc). Så trenger ikke legges i grunt etc.

knutole commented 8 years ago

For å fixe merge conflicts, kan du prøve følgende:

# sørg for å ha add/committed alt på din branch først

git checkout master
git pull origin master
git checkout globesarSlider
git merge master
# merge conflicts dukker opp her. rediger filene og fjern alle `<<<<<` etc.. 
# når alt er fixet, så
git add --all .
git commit -m "merge fix"
git push origin globesarSlider

# nå skal PR på github være oppdatert med "merge fix" commit, og conflicts resolved