-
Notifications
You must be signed in to change notification settings - Fork 9
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
Release 7.0 #1520
Comments
@1t5j0y Found RWB fixes related to locations as well in this release, we might need to update that as well in release notes. Moving it to QA failed just to keep track of it. |
@mahalakshme which card are you referring to? |
@mahalakshme these aren't 7.0 cards. Shouldn't these be in 6.1.7 release notes? |
@1t5j0y the fixes for the cards were released only for RWB environment, and merged to 7.0 release/master. So we didnt add release notes for 6.1.7 since its not for Avni. |
@mahalakshme We should have release notes for all releases and cards should be mentioned in the corresponding release notes. We could probably additionally mention which flavors/envs were released for a specific release in the release notes. |
@1t5j0y agreed I had this concern as well, since the release notes are public, I felt users might get confused. We can discuss in standup tomorrow and decide the way forward. |
@1t5j0y as discussed in standup can you add the release notes for RWB fixes made in 7.0 Also @AchalaBelokar was telling fixes made for 7.0.2/7.0.1 she was unable to find in prerelease - looks like it needs to be merged with 7.1.0 branch as well - can you update the readme as well to indicate this, since we remember to merge with master but not if any other future release branch is cut. I can create a card, if you think its not within the scope of this card. Moving to QA failed to keep track of it. |
@mahalakshme we did not agree on including the non 7.0 cards in the 7.0 release notes Merged 7.0 into 7.1 and master. |
reset role; begin transaction; update users set last_modified_date_time = current_timestamp where is_voided = false; -- spreading is not required as only one user is commit; rollback;
The text was updated successfully, but these errors were encountered: