-
Notifications
You must be signed in to change notification settings - Fork 0
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
Arrow on GPS can no longer be moved #57
Comments
-> according to Puzzle's assessment, the fix for this bug is up to you. I'm contacting you because of feedback from our internal people: When they report a claim and enter the address, it used to be possible to manually determine the position of the claim after an address had been entered. This is apparently no longer possible, presumably since the last update. This is very important for our internal staff, because otherwise they may have to spend a long time looking for the actual location of the damage. |
@paarals can you have a look at this bug please? |
@paarals Can you give me an update till wednesday, 29.11. please? |
Hi @NicolaNaf, can you point me the specific problem?
|
Hi @NicolaNaf You have the problem in here: |
@microstudi thanks for the analysis! I had no idea that you customized the map marker scripts. I have now removed the extra decidim_proposals snippet on the test server now. It must have been a leftover from some bug hunting during a Decidim upgrade. @NicolaNaf does it work better now on test.dialogluzern.ch? |
@microstudi Thx for the analysis. @carlobeltrame : I can enter an address on the test platform (but this was already the case before), but the point on the map still cannot be moved manually. |
That means the additional script was not the real or not the only problem. @microstudi any other ideas? Since this is still a problem occurring in the damage reporting module |
@microstudi: As described by @carlobeltrame , the problem is not with the script. After consultation with Puzzle, we would be happy if you could clarify further what the problem is. Please let us know when we can test the function again. After the problem has been fixed, Puzzle can add the updated code. |
Hi, before we discard is not the script, we should make sure that this script is not duplicated in the site. I've opened the developer tools and I still see both scripts, the proposals one and the reporting proposals. There's something that's adding both when, in the case of the reporting proposals, only the one from the module should be there. That being said, it is possible that there are other problems but we should first discard this one. If you want we could take a deeper look at the whole application, replicate it locally but that is going to take some effort from our part. |
@microstudi Thanks for your fast response. Can you please give me an approximate cost for analyzing and resolving the problem? |
probably a couple/3 of hours worth of time, which implies downloading the code, testing it locally, compiling assets and extract conclusions/make amendments if necessary (a PR). |
@microstudi Alright. Could you provide @carlobeltrame and me with an update by Tuesday morning, December 12th? This will allow me to discuss the results and plan the next steps with Puzzle. |
sure thing, do you want to continue this conversation here or by mail? |
To keep puzzle up to date, I would suggest continuing to communicate via this channel. |
@microstudi Can you please give us an update? |
will do it today, sorry |
hi @NicolaNaf, we found the solution. it was a little bit hard from our side and it took a little bit longer: stadtluzern/decidim-ocl#287. please @carlobeltrame, try if that works. |
@paarals thanks for working on the issue! sounds promising. 👍 @carlobeltrame were you already able to have a look at this? would be great, if you can give us an update tomorrow at our meeting. |
@NicolaNaf I deployed the fix on the test server. Could you test whether it works better now? |
@carlobeltrame We were able to try it out on the test platform and it works again with the manual moving of the arrow for the location. The standard answers (different issue) also works again on the test platform. Since both functions are working again, we would like to make an upload on the poductive plattform. Let's discuss how we want to proceed on our next meeting in 2024. |
@NicolaNaf ist dies eigentlich mittlerweile auf PROD eingespielt? Oder müssen wir nächsten Di besprechen, wie wir damit fortfahren? |
-------- Weitergeleitete Nachricht -------- Liebe Nicole Matthias konnte mir bestätigen, dass es wieder einwandfrei funktioniert. Für unseren heutigen Austausch habe ich keine neuen Themen. Ich wünsche dir eine schöne Woche. Lieber Gruss |
No description provided.
The text was updated successfully, but these errors were encountered: