- Issue created by @lostcarpark
- Merge request !9Cache rebuild needed after adding Verify Email entity. → (Merged) created by nidhi27
- 🇮🇳India nidhi27
Hii,
I have added rebuild and now after saving the entity it is not throwing any error. Let me know if this is the correct way or not.
Thanks! - 🇮🇪Ireland lostcarpark
This looks great and seems to solve the problem.
To get the tests to pass, I had to add the following line in two places:
drupal_flush_all_caches();
In line 76 of
VerifyEmailRouteTest
, and line 62 ofVerifyEmailSetupTest
.I think these can be removed, and providing the tests pass, we should be ready to merge.
- 🇮🇳India nidhi27
Hii,
I have removed drupal_flush_all_caches(); from both the files.
Thanks! - 🇮🇪Ireland lostcarpark
Confirmed the cache flushes have been removed, and tests pass.
I have also tested manually, and run the tests locally, and confirmed all pass.
Great work on this, thanks!