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

[AtmoNouvelleAquitaine] Change description #1423

Merged
merged 2 commits into from
Feb 4, 2020

Conversation

floviolleau
Copy link
Contributor

No description provided.

@somini
Copy link
Contributor

somini commented Jan 12, 2020

LGTM.

I would suggest a minor change:

  • Change the URI to https://www.atmo-nouvelleaquitaine.org/, otherwise the default URI doesn't work. This might require changes to the way the data URL is built to include that prefix /monair/commune/

@floviolleau
Copy link
Contributor Author

Hi,

Thanks for the review. I did the change accordingly

@somini
Copy link
Contributor

somini commented Jan 24, 2020

LGTM, ready to merge.

@teromene teromene merged commit f040e4d into RSS-Bridge:master Feb 4, 2020
@teromene
Copy link
Member

teromene commented Feb 4, 2020

Thank you !

infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this pull request Apr 17, 2020
* [AtmoNouvelleAquitaine] Change description
@floviolleau floviolleau deleted the patch-1 branch May 26, 2020 14:21
@dvikan
Copy link
Contributor

dvikan commented Aug 1, 2023

Details
Type: Error
Code: 0
Message: Call to a member function find() on null
File: bridges/AtmoNouvelleAquitaineBridge.php
Line: 68
Trace
#0 index.php(7): RssBridge->main()
#1 lib/RssBridge.php(18): RssBridge->run()
#2 lib/RssBridge.php(102): DisplayAction->execute()
#3 actions/DisplayAction.php(23): DisplayAction->createResponse()
#4 actions/DisplayAction.php(134): AtmoNouvelleAquitaineBridge->collectData()
#5 bridges/AtmoNouvelleAquitaineBridge.php(39): AtmoNouvelleAquitaineBridge->getIndexMessage()
#6 bridges/AtmoNouvelleAquitaineBridge.php(132): AtmoNouvelleAquitaineBridge->getMaxIndexText()
#7 bridges/AtmoNouvelleAquitaineBridge.php(68)

Context
Query: action=display&bridge=AtmoNouvelleAquitaine&cities=33063&format=Mrss
Version: dev.2023-07-11 (git.master.ed97ce8)
OS: Linux
PHP: 7.4.33 

@floviolleau

@floviolleau
Copy link
Contributor Author

Hi,

I will check it as soon as I have time

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

Successfully merging this pull request may close these issues.

None yet

4 participants