forked from rumkit/PhoneEdit
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Dev #8
Conversation
This file contains 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
* Update PhoneBookContext.db * Update Create.cshtml * Update Delete.cshtml * Update Details.cshtml Co-Authored-By: akinin <[email protected]> * Update Edit.cshtml Co-Authored-By: akinin <[email protected]> * Update Index.cshtml Co-Authored-By: akinin <[email protected]> * Update _Layout.cshtml Co-Authored-By: akinin <[email protected]> * Update site.css Co-Authored-By: akinin <[email protected]> --------- Co-authored-by: akinin <[email protected]>
* Update PhoneBookContext.db * Update Create.cshtml * Update Delete.cshtml * Update Details.cshtml Co-Authored-By: akinin <[email protected]> * Update Edit.cshtml Co-Authored-By: akinin <[email protected]> * Update Index.cshtml Co-Authored-By: akinin <[email protected]> * Update _Layout.cshtml Co-Authored-By: akinin <[email protected]> * Update site.css Co-Authored-By: akinin <[email protected]> * Update PhoneBookController.cs Co-Authored-By: akinin <[email protected]> * Update PhonebookContext.cs Co-Authored-By: akinin <[email protected]> * Update IdentityContext.db Co-Authored-By: akinin <[email protected]> * Update PhoneEdit.csproj Co-Authored-By: akinin <[email protected]> * Update Program.cs Co-Authored-By: akinin <[email protected]> * Update Edit.cshtml Co-Authored-By: akinin <[email protected]> * Update Index action Co-Authored-By: akinin <[email protected]> * Update PhonebookContext.cs Co-Authored-By: akinin <[email protected]> * Create BookEntriesViewModel.cs Co-Authored-By: akinin <[email protected]> * Update PhoneEdit.csproj Co-Authored-By: akinin <[email protected]> * Update Program.cs Co-Authored-By: akinin <[email protected]> * Create PhoneBookService.cs Co-Authored-By: akinin <[email protected]> * Update Create.cshtml Co-Authored-By: akinin <[email protected]> * Update Edit.cshtml Co-Authored-By: akinin <[email protected]> * Update Index.cshtml Co-Authored-By: akinin <[email protected]> * Update _Layout.cshtml Co-Authored-By: akinin <[email protected]> * Scaffold Identity pages Co-Authored-By: akinin <[email protected]> --------- Co-authored-by: akinin <[email protected]>
* Update PhoneBookContext.db * Update Create.cshtml * Update Delete.cshtml * Update Details.cshtml Co-Authored-By: akinin <[email protected]> * Update Edit.cshtml Co-Authored-By: akinin <[email protected]> * Update Index.cshtml Co-Authored-By: akinin <[email protected]> * Update _Layout.cshtml Co-Authored-By: akinin <[email protected]> * Update site.css Co-Authored-By: akinin <[email protected]> * Update PhoneBookController.cs Co-Authored-By: akinin <[email protected]> * Update PhonebookContext.cs Co-Authored-By: akinin <[email protected]> * Update IdentityContext.db Co-Authored-By: akinin <[email protected]> * Update PhoneEdit.csproj Co-Authored-By: akinin <[email protected]> * Update Program.cs Co-Authored-By: akinin <[email protected]> * Update Edit.cshtml Co-Authored-By: akinin <[email protected]> * Update Index action Co-Authored-By: akinin <[email protected]> * Update PhonebookContext.cs Co-Authored-By: akinin <[email protected]> * Create BookEntriesViewModel.cs Co-Authored-By: akinin <[email protected]> * Update PhoneEdit.csproj Co-Authored-By: akinin <[email protected]> * Update Program.cs Co-Authored-By: akinin <[email protected]> * Create PhoneBookService.cs Co-Authored-By: akinin <[email protected]> * Update Create.cshtml Co-Authored-By: akinin <[email protected]> * Update Edit.cshtml Co-Authored-By: akinin <[email protected]> * Update Index.cshtml Co-Authored-By: akinin <[email protected]> * Update _Layout.cshtml Co-Authored-By: akinin <[email protected]> * Scaffold Identity pages Co-Authored-By: akinin <[email protected]> --------- Co-authored-by: akinin <[email protected]>
akinin
pushed a commit
that referenced
this pull request
Jan 23, 2024
* Update UI elements styles in Index.cshtml In the PhoneBook Index view, changes have been made in the style of the badge and buttons to partially fix their appearance. This included modifying the background color for the badge and varying the width and icon dimensions of the Edit, Details, and Delete buttons. * Simplify layout by removing environment-specific CSS and JS The commit removes the distinction between development and other environments for CSS and Javascript in the _Layout.cshtml file. It simplifies the page layout by consistently using local versions of Bootstrap and Font-Awesome, and by modifying the placement of certain elements. All environments will now use the same CSS and JS, which streamlines testing and deployment. * Add check for non-empty model in Index view This commit includes a check to ensure that the list provided to the Index view of the PhoneBook is not empty. Previously, if the list was empty, the view would attempt to take the first entry and fail. The if statement ensures this error is avoided. * Refactor search functionality in PhoneBookController Search string validation was simplified and applied directly to the database query rather than to an in-memory list of entries. Further, the ordering and pagination of entries were also shifted to the query level, improving efficiency and performance. * Refactor database connection setup and enhance options configuration Removed excessive use of `MySqlConnector` and refactored the way connection strings are retrieved. Also applied cleaner structuring for adding database context services. Added configuration for `DefaultUserOptions` and conditional addition of `DatabaseDeveloperPageExceptionFilter` based on the environment.
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.
No description provided.