-
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
LET-1383 [BE] Change tourism-and-recreation for sustainable-tourism #785
LET-1383 [BE] Change tourism-and-recreation for sustainable-tourism #785
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, just take into account that there are references to the old slug in the FE code. unless agreed otherwise, perhaps we could update the FE code as well not to break this?
Good point, I hoped that they don't use it hardcoded anywhere 🤔 . From my point of view, it should be enough to just replace it everywhere on FE |
This PR is BE part for replacing Tourism and recreation for Sustainable tourism (LET-1376) and includes following updates:
Testing instructions
Actual tests are updated and cover the implemented changes (since no new functionality has been introduced).
The results of running new Migration in local database:
before migration
after migration
Tracking
1383