Skip to content

Improve overall code along with automation #38

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
Open
wants to merge 59 commits into
base: master
Choose a base branch
from
Open

Improve overall code along with automation #38

wants to merge 59 commits into from

Conversation

@mehul0810
Copy link
Collaborator

@mehul0810 mehul0810 commented Apr 5, 2024

Description

How Has This Been Tested?

Screenshots (jpeg or gifs if applicable):

Types of changes

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows has proper inline documentation.

mehul0810 added 30 commits June 22, 2021 00:03
@mehul0810
update usernames for essential fiscal hosts
21e9574
@mehul0810
restrict non-admin users to access assets manager
740e975
@mehul0810
improve coding standards
6ed7d6e
@mehul0810
add admin settings header html
6b39540
@mehul0810
improve menu markup
afd8a60
@mehul0810
update settings url and improve code
9d57dae
@mehul0810
update package lock file
bf1aab0
@mehul0810
remove old retina logo
5b39f56
@mehul0810
add new retina logo
ec4b7dd
@mehul0810
load the new logo with img tag
5a80841
@mehul0810
add missing alt tag for the logo image
a58a4e9
@mehul0810
divide the admin settings into sections
a573c94
@mehul0810
remove legacy code from settings page
56dda0c
@mehul0810
revamp the admin setting fields structure
f29e5b3
@mehul0810
add helper fn for admin to get current tab
99a259a
@mehul0810
render the basics of the admin settings fields
9672e11
@mehul0810
improve tabs loading and active status
059d62f
@mehul0810
add fn to render url field for admin
9ec11c5
@mehul0810
add fn to render text field
90aa2d5
@mehul0810
relocate help link call to better place
cf5f850
@mehul0810
add placeholder support to all admin fields
7f725f8
@mehul0810
update readme for changelog
33ae486
@mehul0810
add fn to render action for admin settings
397550a
@mehul0810
add and load check our documentation sidebar section
984e3bb
@mehul0810
open in new tab for check our documentation image
bf649dc
@mehul0810
improve save wrap section
ce0978a
@mehul0810
improve text display on render action fn
d2fdee5
@mehul0810
add id to the submit btn
6d23823
@mehul0810
add admin js to webpack
542ff51
@mehul0810
add and update package json and its lock file
be131ce
mehul0810 and others added 23 commits August 26, 2021 16:22
@mehul0810
add helper fn to get settings
7f5908a
@mehul0810
add save_settings fn
597efd3
@mehul0810
improve composer and its lock file
4493e68
@mehul0810
improve save_settings fn
11736b9
@mehul0810
store textarea data as array
7afeeea
@mehul0810
display stored value for each field type
feb942a
@mehul0810
resolve array to string conversion error
ad37696
@mehul0810
fix checkbox checked unchecked value
04807d4
@mehul0810
save admin settings via ajax
dccdc7d
@mehul0810
add missing dns prefetch support
cb9259b
@mehul0810
add missing preconnect support
57bbfb2
@mehul0810
improve storing tactic for textarea
71c5ded
@mehul0810
add helper fn to compress html
0ba780d
@mehul0810
optimize preconnect and dns prefetch
e7801ae
@mehul0810
improve fetching settings for basic
e58ff7b
@mehul0810
fix undefined index error
b5b72a9
@mehul0810
don't load assets manager if not enabled
299018b
@mehul0810
update assets manager enable settings
9040af0
@mehul0810
ui touchup for assets manager
56b1ae6
@mehul0810
add global mixins file
5911e78
@mehul0810
improve assets manager hmtl markup
3c2d1b7
@mehul0810
import global mixins file to frontend
81c0d0e
add automation files
1a63ebf
@github-actions
Copy link

github-actions bot commented Apr 5, 2024 •
edited
Loading

Test on Playground
Test this pull request on the Playground

@mehul0810 mehul0810 changed the base branch from develop to master November 12, 2024 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mehul0810