-
Notifications
You must be signed in to change notification settings - Fork 23
TASK: Sync dev to master #1617
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
Merged
Merged
TASK: Sync dev to master #1617
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
…bel-1579 ENH: Change phrasing of "Subscribe to Topics"
Refactor settings and modernize codebase Refactored settings management by replacing `SettingsInfo` with the new `ModuleSettings` class, streamlining handling of module settings. `SettingsInfo' is now used to manage feature settings Updated database schema to rename `GroupKey` to `SettingsKey` and removed obsolete functions and procedures. Introduced `SettingsController` for structured CRUD operations. Updated token replacement and template handling to align with the new settings structure. Modernized code with inline property initializers, string interpolation, and improved readability. Incremented module version to `09.02.00` and updated `.dnn` manifest files. Removed deprecated SQL functions and procedures. Updated unit tests and UI components to use `ModuleSettings`. Enhanced release notes and documentation to reflect changes.
TASK: Refactor settings and modernize codebase
…r-on-move-1594 FIX: Error moving topic to another forum
Fix bad SQL merge in #1595
TASK: Add/Updates Web API controllers
TASK: Correct/standardize user's TopicCount/ReplyCount #1599
TASK: Create database objects to support badges #1480 TASK: Create UI for administrator to manually assigned badges #1485 TASK: Badge assignment process #1484 TASK: Add badge metric name to control panel badge list #147 TASK: User profile UI to show badges #1486 TASK: Add badge management to control panel #1483 TASK: Create default badges during install or upgrade #1488 TASK: Badge token replacement logic #1482 TASK: Add BadgeId to Process queue #1487 TASK: Add badge notification option to user profile #1486 TASK: Manual badge assignment UI for moderators #1485 TASK: Web API seucrity checking "ForumsAuthorizeAttribute" for certain functions (e.g. Ban) should not require forumId to be passed to the API #1507
…5-1486-1487-1488-1535-1599' of https://github.com/johnhenley/Dnn.CommunityForums into dev92/2-badges-48-427-1479-1480-1481-1482-1483-1484-1485-1486-1487-1488-1535-1599
…480-1481-1482-1483-1484-1485-1486-1487-1488-1535-1599 ENH: Implements user badge feature
…s-key-1547 TASK: Remove colon ":" from SettingsKey
…e-net-1608 FIX: Datatables.NET paging
Encode non-latin characters in content-disposition
…ct-1607 FIX: HTML Decode Email notification subject
…en in Social Groups mode, and if not, remove any settings associated with social group
…ttings-1606 Dev92/social group module settings 1606
…1593 FIX: Correct resource key for Control Panel tag display
TASK: Release Prep
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.
Description of PR...
Sync dev to master