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

[NextBus] SFMTA not supported #122286

Closed
drozycki opened this issue Jul 21, 2024 · 6 comments
Closed

[NextBus] SFMTA not supported #122286

drozycki opened this issue Jul 21, 2024 · 6 comments

Comments

@drozycki
Copy link
Contributor

The problem

I'm trying to use the NextBus integration to show SFMTA arrival times near my home. NextBus (now UmoIQ) supports the SFMTA (link), but the SFMTA does not show up in the agency dropdown when configuring the integration! This is probably because the agencyList API call omits this (and many other) agencies. There used to be a workaround (specify agency tag sfmta-cis in configuration.yml) but support for that was dropped by this PR three months ago!

What version of Home Assistant Core has the issue?

core-2024.7.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

NextBus

Link to integration documentation on our website

https://www.home-assistant.io/integrations/nextbus/

Diagnostics information

No response

Example YAML snippet

sensor:
  - platform: nextbus
    name: "Outbound K"
    agency: sfmta-cis
    route: K
    stop: "6991"

Anything in the logs that might be useful for us?

No response

Additional information

I have raised this issue at info@cubicsystems.com (NextBus/UmoIQ parent company) but frankly I am not confident that they will fix their API anytime soon. I also reached out to the SFMTA.

@home-assistant
Copy link

Hey there @ViViDboarder, mind taking a look at this issue as it has been labeled with an integration (nextbus) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of nextbus can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign nextbus Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


nextbus documentation
nextbus source
(message by IssueLinks)

@drozycki
Copy link
Contributor Author

CC'ing the author and reviewer of the PR which broke the workaround @jpbede @joostlek

Why was configuration.yaml support removed? Can we revert that change?

I suppose we could hardcode SFMTA in config_flow.py: _get_agency_tags if you prefer.

I'd be happy to whip up a PR whatever is decided.

@joostlek
Copy link
Member

Apparently APIs changed and a PR is open to recruit the whole thing

@drozycki
Copy link
Contributor Author

#121133

@drozycki
Copy link
Contributor Author

Thanks for merging the fix to dev last week. I noticed the commit was not included in 2024.7.4. Do you expect it to release in 2024.8.0 next week?

@drozycki
Copy link
Contributor Author

drozycki commented Aug 8, 2024

I confirmed that the issue is resolved in today's 2024.8.0 release. Thanks all for getting this out so quickly!

@drozycki drozycki closed this as completed Aug 8, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Sep 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants