Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajouts #12

Closed
wants to merge 3 commits into from
Closed

Ajouts #12

wants to merge 3 commits into from

Conversation

iambluedev1
Copy link
Contributor

  • j'avais un bug au niveau de la requête curl en localhost qui ne s'effectuait pas. Cette petite modification l'a réglé.
  • ajout de la possibilité d'augmenté la limitation au niveau de l'api github

@Eywek
Copy link
Member

Eywek commented Apr 25, 2018

Merci pour les identifiants Github mais ça va être drop au final, je vais utiliser https://raw.githubusercontent.com/MineWeb/mineweb.org/gh-pages/market/plugins.json donc plus de restriction sur l'API de Github. Donc au final seul le premier commit est nécessaire

EDIT: cf. #26

@iambluedev1
Copy link
Contributor Author

iambluedev1 commented Apr 25, 2018

Ah okep, et pour la mise à jour du cms ça passera toujours par l'archive de la branche master ? Parce que j'ai dév un système qui va seulement faire les actions nécessaires (ajouter, supprimer, modifier) grâce à la comparaison entre deux releases, vu que pour le moment ça dl toute l'archive, et tous les fichiers sont déplacés. Donc du coup pour savoir si c'est utile que je fasse un pr.

@Eywek
Copy link
Member

Eywek commented Apr 28, 2018

Pour l'instant yep je me base du master mais si tu veux améliorer le système avec un système de comparaison des releases yep ça peut être pas mal !

@Eywek Eywek self-requested a review April 28, 2018 13:52
$ch = curl_init();
curl_setopt($ch, CURLOPT_URL, $url);
curl_setopt($ch, CURLOPT_HTTPHEADER, [
'User-Agent: MineWebCMS'
]);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($ch, CURLOPT_FOLLOWLOCATION, 1);
curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, 0);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seul cette ligne est donc utile

@iambluedev1
Copy link
Contributor Author

okep bah je vais faire ça

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants