Two mails in Inbox instead of one (2.2.1)

[martindusinberre]

On 2.2.1, mail1.json generates two emails in the Inbox. It should only be one.

[christian-bernet]

Estimate 2 hours

[christian-bernet]

we can not reproduce this situation. Maybe there was an error in the content that is now away.
@martindusinberre can you please verify.

[martindusinberre]

Hi Christian: please reassign this to me once #51 has been fixed, and then I can test for the two-email problem on 2.2.1 again.

[christian-bernet]

ok. Assigned now to you. Please test now again the email problem on 2.2.1.

[martindusinberre]

Sorry, the bug is still there.

[christian-bernet]

This situation is because of corrupt content after importing it with the importer. It is not because of wrong .json files. These are ok.
So this is also a result of the unstable importer.

Because this is wasting time for searching each time on such corrupt content situations (we had this situation already two or three times), we decided to change the importer to a simple concept like this:

• For test-system with flag deletePlayerState = true: insert new topic and delete previous imported topic and delete players state
• for production with flag deletePlayerState = false: insert new topic and deactivate last imported topic

[christian-bernet]

that is still open. If you do a second import it still creates two mails!
This is an importer bug. I will check that after my vacation.

[christian-bernet]

can not reproduce it locally any more at this time. But importer needs still a bit of code changes.

[christian-bernet]

Title of page 2.2.1 is "Chapter 2/5: Back home"
(ask prof. for more time, to get there)
Bug is still there.

[christian-bernet]

This bug ist still here:

[interrogator]

@christian-bernet if and when this gets fixed, please let me know so I can version bump and redeploy livesintransit.org...

[martindusinberre]

Hi Christian: great, thanks for the update. @lfeine could you please test this when you have time? Best, Martin

[interrogator]

@christian-bernet are the changes related to this issue all in master branches? If so, this issue can be closed, as changes will go into next deployment.

[christian-bernet]

@interrogator We did not change anything this week.
So, yes. this changes are all in master branch.

[lfeine]

I'm not sure the previous messages mean that this still needs testing, but it doesn't show up to me anymore.

Unfortunately, when replying to the formerly duplicated E-Mail, the reply button appears to not do anything (no state changes, no loading bar). The mail is still sent however, and appears when swapping back from another mail or closing and opening the mail window.

The error that appears in the console after pressing the reply button is:
Error: Request failed with status code 400 app.js line 628 > eval:16:15 (Firefox 71.0)

[interrogator]

@christian-bernet

[christian-bernet]

I had a quick look on it.

There is an auto page transition when you reply to that mail which will fail, if you are not on page 2.2.1.
I can fix that on Monday. Unfortunately I can't do it today because we reserved the day for you last Monday.

@martindusinberre Does that work for you?

[martindusinberre]

@christian-bernet I had a look at this and I think it's a problem in the backend, right, not in the JSON files that I have. If so, then yes, please fix. The idea is that you shouldn't be able to transition from 2.2.1 to 2.2.2 unless you answer the email 2.2.1/mail1.json
Thanks, Martin

[christian-bernet]

Bug for mail reply fixed: d3ff4ad
@martindusinberre but you have to change the jsons a litte bit, if the page transition from 2.2.1 to 2.2.2 should only be available, if you reply to the mail on 2.2.1.
You have to remove the pageTransition in mail1.json on page 2.2.1 and it will work.
With the pageTransition in mail1.json an automatic pageTransition after you reply to the mail will happens. But to have that on the reply of the same mail is not possible at the moment. Technically this gives a circle relationship, which the system at the moment can not handle.

Please let me know, if it is now ok for you like this.
All best, Christian

[christian-bernet]

We will have again a look at this tomorrow and see if we can fix the problem with the circle relationship for pagetransition on page 2.2.1

[martindusinberre]

Many thanks, Christian. I'm not sure why there's a particular problem here, as it should be a similar functionality as on 2.2.8, where you can only transition to 2.2.9 if you answer the previously received email (2.1.6). Here's it's always functioned fine, so I'm a bit surprised if the transition from 2.2.1 to 2.2.2, dependent on answering 2.2.1/mail1, doesn't work. Or have I misunderstood? Anyway, let me know if it's faster to talk by phone about this tomorrow--I'm around all day after about 9am.

[nemtish]

Hi @martindusinberre for this kind of situations where user should be transitioned automatically when replies to mail, you have to change "pageTransition.json" file like below:
"affectedMail": null,
"buttonText": null

With this change button will stay invisible for the user and circle relationship (@christian-bernet mentioned) won't be problem

Pages where you have page transitions that are only unlocked (without auto-transition) when user reply to specific mail you can leave like it is

[christian-bernet]

Unfortunately we can not change the content ourself, because we do not have access to the content git anymore.
Add for the page 2.2.8 there is no auto-transition (if user reply to the mail) active.
Auto-transition can be activated if you add the pagetranstion in the mail.
As it is for mail1 on page 2.2.1.