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

Bump pyprosegur to 0.0.13 #138960

Merged
merged 1 commit into from
Feb 20, 2025
Merged

Conversation

dgomes
Copy link
Contributor

@dgomes dgomes commented Feb 20, 2025

Breaking change

Since February 20th Prosegur is actively blocking HomeAssistant user-agent (and this user-agent alone), this upgrade pins the user-agent to something else.

Proposed change

Full Changelog: dgomes/pyprosegur@0.0.9...0.0.13

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New integration (thank you!)
  • New feature (which adds functionality to an existing integration)
  • Deprecation (breaking change to happen in the future)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist
  • I have followed the perfect PR recommendations
  • The code has been formatted using Ruff (ruff format homeassistant tests)
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • The manifest file has all fields filled out correctly.
    Updated and included derived files by running: python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt.
    Updated by running python3 -m script.gen_requirements_all.
  • For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.

To help with the load of incoming pull requests:

Copy link
Contributor

@edenhaus edenhaus left a comment

Choose a reason for hiding this comment

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

Thanks @dgomes 👍

@edenhaus edenhaus merged commit 9cbed48 into home-assistant:dev Feb 20, 2025
63 of 64 checks passed
frenck pushed a commit that referenced this pull request Feb 21, 2025
@frenck frenck mentioned this pull request Feb 21, 2025
@Santiago-DelBono
Copy link

After the update now you can arm, disarm but I am not able to pull the control panel state. So, if your automatizations depends on Prosegur state, unfortunately will not work.
Screenshot_20250222_004641_Home Assistant

@lazynoda
Copy link

It works like a charm, for now 👌

@Santiago-DelBono
Copy link

It works like a charm, for now 👌

Not really, the status keep as unknown, you can arm and disarm but you are not able to see the state of the alarm

@dgomes
Copy link
Contributor Author

dgomes commented Feb 22, 2025

There is a new fix up for merge (you need to wait for release )

@Santiago-DelBono
Copy link

There is a new fix up for merge (you need to wait for release )

Ohh, great to know that. Thanks for your work. Cheers

@lazynoda
Copy link

It works for me 🤷‍♂️ status and buttons to change mode

@Santiago-DelBono
Copy link

It works for me 🤷‍♂️ status and buttons to change mode

From what country? What URL you use to login?

@lazynoda
Copy link

Spain. And I don't use any URL, just the app

@Santiago-DelBono
Copy link

Spain. And I don't use any URL, just the app

I mean the URL when you usually login to Prosegur

@github-actions github-actions bot locked and limited conversation to collaborators Feb 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integration "Movistar Prosegur" not working
5 participants