-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Fixed ticket_6217 #6245
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
Fixed ticket_6217 #6245
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.
Thanks for the PR, there are more places where the flag should be removed. You should search in all the code base.
ok..so now should I close this pull request ? |
No, you should push your new changes on this branch |
ok, then I need to create new pull request ? |
No, this pull request is updated when you push new changes. You should check https://opensource.guide/how-to-contribute/#opening-a-pull-request |
Here, I am again tried to remove 'Search_Analytics' Flag. Can you give review on this ? |
url( | ||
r'^(?P<project_slug>[-\w]+)/search-analytics/$', | ||
SearchAnalytics.as_view(), | ||
name='projects_search_analytics', |
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.
Only the flag should be removed, not the functionality
can you review it ? |
This was done in #6447 |
I am new towards contributions and I have tried to solve this problem.