-
-
Notifications
You must be signed in to change notification settings - Fork 564
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
Finalize renaming adlists -> lists #2756
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
…oard. This fixes a few broken links such as "Manage lists" on the dashboard as well as the generated links from the Search Lists page to the Groups -> Lists page with enabled list row highlighting Signed-off-by: DL6ER <[email protected]>
73dfcc5
to
7e879b2
Compare
Conflicts have been resolved. |
I'm not so happy with that change. I know gravity can now contain also allowlists (anti-gravity) and adlists is not the best term anymore. However, a generic 'list' makes it impossible to distinguish gravity from manually managed allow/deny lists. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. |
Would you mind expanding on this a little? Do you have an alternative suggestion? |
Maybe along the lines of |
In the end it comes down to the difference between |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Coming back to this... I think |
Conflicts have been resolved. |
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.
In groups-lists.lp, line 65, consider changing "List of subscribed lists" to "Subscribed lists"
What does this implement/fix?
Finish the only partially done renaming
adlists
->lists
on the dashboard. This fixes a few broken links such as "Manage lists" on the dashboard as well as the generated links from the Search Lists page to the Groups -> Lists page with enabled list row highlightingRelated issue or feature (if applicable): N/A
Pull request in docs with documentation (if applicable): N/A
By submitting this pull request, I confirm the following:
git rebase
)Checklist:
developmental
branch.