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

Image url show internal pasteur URL instead of connect's #47

Open
r03ert0 opened this issue Mar 15, 2021 · 3 comments
Open

Image url show internal pasteur URL instead of connect's #47

r03ert0 opened this issue Mar 15, 2021 · 3 comments
Labels
bug Something isn't working

Comments

@r03ert0
Copy link
Member

r03ert0 commented Mar 15, 2021

The images uploaded by the user have the internal pasteur url (https://ghfc17.hosting.pasteur.fr/...). This should be replaced by the ma-voix url: https://mavoix.connect-project.io/...

@r03ert0 r03ert0 added the bug Something isn't working label Mar 15, 2021
@ntraut
Copy link
Member

ntraut commented Apr 7, 2021

It's because I made the config at the time when we did not have https://mavoix.connect-project.io/. Now it should be fixed.

@ntraut
Copy link
Member

ntraut commented Apr 26, 2021

The Parse objects TabItem and Asset still contain the ghfc17 url since they are stored with it when they are created. The url is updated by the app when it receives the object so the request is now made to the mavoix url. I think we could avoid that translation but we would have to change the way TabItem and Asset objects are stored.

@r03ert0
Copy link
Member Author

r03ert0 commented Apr 26, 2021

Yes. Better do these basic structural changes now, that we don't have any real users. If needed, we can just erase the whole DB.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants