Promoting to prod #592
Merged
Promoting to prod #592
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…y in Safari -- Fixed. Reason was the missing # in URL. # is required for angular routes.
…fari Github issue #566, Connect Login/Logout redirect not working correctly in Safari
-- Implemented the required behaviour with Discussions page. Took liberty to implement the same behaviour when user has unposted content and tries to change the thread from the left panel of thread list.
-- Implemented the required behaviour with Dashboard page. -- Tried to handle the warning of multiple leave hooks for the same route, however, the solution didn't work. Moving ahead as right now it is a warning and not causing any problem. May be we can launch a challenge to get it fixed.
…d_content_alert Feature/discussions unposted content alert
…y in Safari -- Fixed logout issue in Safari
…fari Github issue #566, Connect Login/Logout redirect not working correctly in Safari
…ox using (x) button, Can add the user by clicking the ADD button -- Removed X icon for IE 11
…ar_icon_ie11 Github issue #360, Even though user clear the username from the textbox using (x) button, Can add the user by clicking the ADD button
Updating footer Help link to point to Help Center FAQ instead of general help page
Updating footer Help link to point to new Connect FAQ instead of general help page
Updating footer Help link
Updating footer Help link
Switch the Beta logo with the original Connect logo
… "manager" role -- Aliased administrator roles to act like as connect manager
-- Fixed edge case where a reply to a topic was not being persisted after loading more posts for a topic
…d_content_alert Github #166, Messaging: unposted content alert
-- Added footer for the discussion page
…ussions Github issue #574, Discussions: no footer on Discussion tab
-- Adjusted height of the left panel to accommodate the footer.
…ussions Github issue #574, Discussions: no footer on Discussion tab
-- Cleared the redux state on successful projects load action
… update -- Removed the constraint of user to be a team member of the project in order to view the comments of status updates. Further, I am assuming we need to do the same for Discussions tab as well, so removing the constraint there as well. So, now any user who has the access to the project's dashboard and discussions pages, can view posts and comments.
… update -- Reverted previous changes. Found the root cause of the problem. Actually the flag, to prevent non team member from commenting on posts, was wrongly being used to render the feed's comments section on dashboard. It is used correctly on Discussions page.
…triction_being_memeber Github issue #572, Messaging: Owner response post missing from status update
Github issue #568, Dashboard: status update thread is too "sticky"
…nager Github issue #579, General: allow "admin" role to have same rights as "manager" role
added 8 commits
December 15, 2016 12:21
-- Fixed rendering issue of the footer for the discussions tab. Thanks @vic-appirio
….topcoder.com -- Added redirect to connect from connectv2
…ussions Github issue #574, Discussions: no footer on Discussion tab
…connect Github issue#561, General: Redirect connectv2.topcoder.com to connect.topcoder.com
-- Fixed. We were missing sorting the posts when loading more posts for a feed
-- Ignored empty posts whose author are 'system'
…_posts Github issue#585, Ignore 'user-joined' type posts in Topic comments
Github issue#587, Comments Sort order within a topic is incorrect
vikasrohit
requested review from
parthshah
and removed request for
parthshah
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Github #166, Messaging: unposted content alert
Github issue #579, General: allow "admin" role to have same rights as "manager" role
Github issue #574, Discussions: no footer on Discussion tab
Github issue #568, Dashboard: status update thread is too "sticky"
Github issue #572, Messaging: Owner response post missing from status update
Github issue #561, General: Redirect connectv2.topcoder.com to connect.topcoder.com
Github issue #587, Comments Sort order within a topic is incorrect
Github issue #585, Ignore 'user-joined' type posts in Topic comments
Updated footer links
Removed beta from logo
@parthshah @fnisen @ajefts fyi