[Sprint/LuckyLizard](fix) Boost console feed #1596
Closes #1596 (closed)
Depends on: engine!311 (merged)
Try the create boost button for the different pages here https://fix-boost-console-feed-1596.minds.io/boost/console/newsfeed/create
added Squad::Green scoped label
added MR::Awaiting Review scoped label
- Resolved by Ben Hayward
mentioned in issue #1596 (closed)
marked as a Work In Progress
changed the description
unmarked as a Work In Progress
- Developer
I tested this and it looks good. Once you address Mark's comment above, I will give it my approval.
Edited by Brian Hatchet resolved all threads
- Developer
Cool, we should be ready to go if you'd be so kind as to quickly test again
added 53 commits
- 2a4f4f00...5ed6e576 - 52 commits from branch
master
- 9a219148 - Merge remote-tracking branch 'origin/master' into fix/boost-console-feed-1596
- 2a4f4f00...5ed6e576 - 52 commits from branch
approved this merge request
unapproved this merge request
approved this merge request
- Resolved by Ben Hayward
added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label
added 1 commit
- 704793e8 - Updated to pull sidebar from v2 feeds endpoint
added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label
resolved all threads
resolved all threads
added 12 commits
- 704793e8...265cc9d3 - 11 commits from branch
master
- 3b97a797 - Merge branch 'master' of gitlab.com:minds/front into fix/boost-console-feed-1596
- 704793e8...265cc9d3 - 11 commits from branch
Hi Minds Devs, good morning.
@benhayward.ben Just would like to follow-up if there is a possibility for issue this getting fixed?
Thank you for all your efforts.
All the best.
added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label
- Developer
Found 2 issues in this @Zoraya, so both will need to be fixed, and are on my cards to do shortly. No worries! Apologies for the delay.
Thank you, @benhayward.ben. Have a great week :)
changed the description
added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label
assigned to @javanick and @brianhatchet
Thank you for updating this despite the workload @benhayward.ben
I hope this gets fixed soon :)
Good day!
- Developer
Any time, turned out to be quite the large change in the end.
approved this merge request
approved this merge request
mentioned in commit 4cfc63bc
merged