Skip to content

/ phpminiadmin

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

Minimal changes this time, meta no index, copy for 2018, user field r… #33

Open
wants to merge 3 commits into
base: master
from
Open

Minimal changes this time, meta no index, copy for 2018, user field r… #33

wants to merge 3 commits into from

Conversation

@gnovaro
Copy link

gnovaro commented Aug 23, 2018

…equired

This time don't touch project url, fields order or default values
@gnovaro
Minimal changes this time, meta no index, copy for 2018, user field r…
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
28e0989 
…equired

This time don't touch project url, fields order or default values
@JustinStolle
Copy link
Contributor

JustinStolle commented Aug 23, 2018

Notes:

  • Version number should be in this format 1.9.YYMMDD, such as 1.9.180823
  • Password could technically be empty, so making it a required input is probably not helpful
  • The robots exclusion protocol can be handled with a separate robots.txt file maintained by the site owner
  • Removing so many extra line breaks makes the code harder to read
gnovaro added 2 commits Aug 23, 2018
@gnovaro
Fix version
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
7424e72
@gnovaro
Update phpminiadmin.php
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
b90f972
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
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@gnovaro @JustinStolle
You can’t perform that action at this time.