Skip to content
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

Implemented new resource manager as per feedback #5240

Merged
merged 131 commits into from
Aug 30, 2022

Conversation

valadas
Copy link
Contributor

@valadas valadas commented Aug 30, 2022

This new version of the resource manager uses WebComponents. It implements the requested UI/UX improvements requested by many. It no longer depends on the React common components.

If you want to test this, you can download the build result from DevOps and do a clean install.
I would also love some testing on upgrades from 9.10.2 to 9.11.0.
Would love for people using Group/User mode to give this a test too.

Resolves #4214
Resolves #4338
Resolves #4611
Resolves #4612
Resolves #5227

valadas and others added 30 commits February 16, 2022 21:10
Implementented basics of cardview
Ensure folder list children are collapsed when parent is collapsed
Implemented more list-view columns
Switched to non-beta dnn-elements v0.14.0
Switched to non-beta dnn-elements v0.14.0
- Adds a green checkmark feedback on successful uploads
- Makes the long filenames wrap in gridvriew
Polished some UI for uploads and filenames.
@valadas valadas added this to the 9.11.0 milestone Aug 30, 2022
Copy link
Contributor Author

@valadas valadas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few things in the manifest

DNN Platform/Modules/ResourceManager/ResourceManager.dnn Outdated Show resolved Hide resolved
DNN Platform/Modules/ResourceManager/ResourceManager.dnn Outdated Show resolved Hide resolved
SolutionInfo.cs Outdated Show resolved Hide resolved
Copy link
Contributor

@mitchelsellers mitchelsellers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

THANK YOU!!!

@valadas
Copy link
Contributor Author

valadas commented Aug 30, 2022

THANK YOU!!!

@valadas
Copy link
Contributor Author

valadas commented Aug 30, 2022

I'll take a better look at the build issues tomorrow (well today)...

Copy link
Contributor

@donker donker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

@donker donker merged commit c709cf7 into dnnsoftware:release/9.11.0 Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants