lenra-io / server

GNU Affero General Public License v3.0
6 stars 0 forks source link

fix: Consent Page app name #530

Closed jonas-martinez closed 5 months ago

jonas-martinez commented 5 months ago

About this PR

Closes https://github.com/lenra-io/server/issues/527

This was just a bug in the way we match on the env_id. As you can see in this PR, there was a is_binary check on the environment but this could never match because the env_id is an integer.

I change the is_binary to a is_integer and now this works properly.

How to test my changes

You can run the app-molny with a local setup of the server using this branch and you will see on the consent page that the app name is properly displayed.

Checklist

I included unit tests that cover my changes

I added/updated the documentation about my changes

taorepoara commented 5 months ago

:tada: This PR is included in version 1.3.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket: