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.

Sign up for GitHub

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

Jump to bottom

Use AJAX for Categories admin #381

Merged
merged 6 commits into from
Apr 13, 2020
Merged

Use AJAX for Categories admin #381

merged 6 commits into from
Apr 13, 2020

Conversation

EdiWang
Copy link
Owner

@EdiWang EdiWang commented Apr 13, 2020

Address #376

@EdiWang EdiWang requested a review from Anduin2017 April 13, 2020 08:37
@EdiWang EdiWang self-assigned this Apr 13, 2020
@EdiWang EdiWang merged commit 493bc4f into master Apr 13, 2020
@EdiWang EdiWang deleted the cat-ajax-admin branch April 13, 2020 08:49
@Anduin2017
Copy link
Collaborator

Ajax is considered bad because you can't use it without javascript.

@EdiWang
Copy link
Owner Author

EdiWang commented Apr 13, 2020

Ajax is considered bad because you can't use it without javascript.

But it is still better than refreshing the page just for CRUD an entity in MVC...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Anduin2017 Anduin2017 Awaiting requested review from Anduin2017

Assignees

@EdiWang EdiWang

Labels
area-admin area-category UI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@EdiWang @Anduin2017