1132 replies to this topic

[cobus]

Idd werkt goed.

[Loui (PD1LOI)]

Plugin het weer zit toch gewoon in de e2openplugins?

Klopt Eric en ik hou die bij normaal, maar nu Buienradar de codes zo heeft veranderd dat hier niks meer mee gedaan kan worden.

Wat mij betreft haal HetWeer eruit en zet WeerInfo er weer in.dan is ieder weer up to date.

[Erik Slagter]

Kun je niet hetweer niet aanpassen met de code uit de plugin die nu aangeboden wordt?

 

Ik ben ab-so-luut niet voor 3rd party packages.

[cobus]

Erik dit ga je toch niet menen hé? Is er iemand eindelijk mee bezig en compleet vanaf 0 opgebouwd ga jij dit vragen sorry hoor dat vind ik dus absoluut niet gepast.

[cobus]

Komt ook nog bij dat je eventueel overleg kunt plegen met de maker om hem erin te zetten zo ja hoe hij dat moet aanpakken.

[Rob van der Does]

Wat ik wel gepast vind, is het verzoek de plugin te plaatsen in de 'open plugins'. Dan kan er gebouwd worden vanuit de sources (wat altijd veel eleganter is dan het plaatsen van een IPK op de feeds) en het geeft een ieder gemakkelijk de mogelijkheid pull-requests in te dienen.

[Tech]

En makkelijker te onderhouden.

[Rob van der Does]

I created a pullrequest: https://github.com/E...-HetWeer/pull/1

 

When this has been accepted I would kindly ask Caugth to push his updates to the repos.

[Loui (PD1LOI)]

I created a pullrequest: https://github.com/E...-HetWeer/pull/1

 

When this has been accepted I would kindly ask Caugth to push his updates to the repos.

Voor mij geen probleem om het aan Caught over te dragen Pullrequest  geaccepteerd. Dat ik daar niet eerder aan gedacht heb.

Van nu af aan gewoon te downloaden via de normale weg zoals voorheen.

Edited by Loui (PD1LOI), 11 September 2016 - 17:39.

[Rob van der Does]

Oh, sorry, ik zie dat ik naar Engels ben overgeschakeld

[Loui (PD1LOI)]

Oh, sorry, ik zie dat ik naar Engels ben overgeschakeld

Geen probleem,maar goed dat je dat gedaan hebt Rob.


Verstuurd vanaf mijn One X met Tapatalk

[Caught]

I created a pullrequest: https://github.com/E...-HetWeer/pull/1

 

When this has been accepted I would kindly ask Caugth to push his updates to the repos.

Zeg voor mij geen probleem hoor, maar voor mij is dit de eerste keer dat ik github zou gebruiken.

Dus ik ga het nog even moeten uitvogelen. Al begrijp ik dat het werk voor nu, er reeds opzit?

 

Btw. Ik volg steeds hoor, maar ik was vooral bezig met een ander projectje die veel tijd heeft gekost.

[Loui (PD1LOI)]

 

I created a pullrequest: https://github.com/E...-HetWeer/pull/1

 

When this has been accepted I would kindly ask Caugth to push his updates to the repos.

Zeg voor mij geen probleem hoor, maar voor mij is dit de eerste keer dat ik github zou gebruiken.

Dus ik ga het nog even moeten uitvogelen. Al begrijp ik dat het werk voor nu, er reeds opzit?

 

Btw. Ik volg steeds hoor, maar ik was vooral bezig met een ander projectje die veel tijd heeft gekost.

 

Als het goed is heb je toegang tot Github hier de link er naar toe https://github.com/P...nplugin-HetWeer

[Rob van der Does]

 

I created a pullrequest: https://github.com/E...-HetWeer/pull/1

 

When this has been accepted I would kindly ask Caugth to push his updates to the repos.

Zeg voor mij geen probleem hoor, maar voor mij is dit de eerste keer dat ik github zou gebruiken.

Dus ik ga het nog even moeten uitvogelen. Al begrijp ik dat het werk voor nu, er reeds opzit?

Je laatste versie staat nu op github. Het zou dus fijn zijn, als je komende aanpassingen daar wilt aanbrengen.

Als je hulp nodig hebt met het gebruik van github, zou ik je via Skype kunnen helpen (Skypename = robvanderdoes).

[Rob van der Does]

Wat ik alleen niet begrijp, is dat de plugin 'HetWeer' niet voorkomt in https://github.com/E...s/e2openplugins maar wel in https://github.com/E2OpenPlugins

[Rob van der Does]

Zojuist geüpdatet, en de nieuwe 'hetweer' staat inderdaad online. Ondanks:

Wat ik alleen niet begrijp, is dat de plugin 'HetWeer' niet voorkomt in https://github.com/E...s/e2openplugins maar wel in https://github.com/E2OpenPlugins[/font][/size]

Kan iemand dat uitleggen?
 
En er is één dingetje fout gegaan: de in de plugin opgenomen 'plugin.png' wordt niet vertoond; heb ik die niet op de goede manier ingebracht?

[Loui (PD1LOI)]

Zojuist geüpdatet, en de nieuwe 'hetweer' staat inderdaad online. Ondanks:

Wat ik alleen niet begrijp, is dat de plugin 'HetWeer' niet voorkomt in https://github.com/E...s/e2openplugins maar wel in https://github.com/E2OpenPlugins[/font][/size]

Kan iemand dat uitleggen?
 
En er is één dingetje fout gegaan: de in de plugin opgenomen 'plugin.png' wordt niet vertoond; heb ik die niet op de goede manier ingebracht?

 

Ik heb het aangepast nu gaat het wel goed. Maar er gaat meer mis Caught moet in de plugin het pad veranderen van WeerInfo naar Het Weer. Het enige wat je nu nog kan doen zodat het werkt is de map "Images" in de map Weerinfo plaatsen. bestaat die map nog niet even aanmaken in Extentions.

Ik kan natuurlijk ook in Github de naam veranderen naar WeerInfo, maar ik weet niet of het dan bij OpenPli goed gaat.Oplossing = een van de twee moet hernoemd worden.

[Loui (PD1LOI)]

Het eenvoudigst lijkt mij in github de naam veranderen,dan is er voor Caught verder niks te veranderen.Hij heeft al werk genoeg hi

Edited by Loui (PD1LOI), 12 September 2016 - 08:29.

[Rob van der Does]

Sorry Loui, ik kan nu niet meer volgen wat je allemaal gedaan hebt. er staat ook geen enkel logische commit tekst bij.
Het veranderen van het pad voor de images was een fluitje van een cent geweest.
Het aanpassen van de naam van de plugin heeft meer voeten in de aarde.
Is het goed als ik je commits van vandaag revert en van daaruit de plugin weer in orde maak?

[Rob van der Does]

Ik heb een pullrequest gemaakt met alle noodzakelijke wijzigingen.

Als Caught een momentje heeft, kan hij de auto-update er nog even volledig uithalen; dat lukte mij niet.