Skip to content

Rjh branch #2

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

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Rjh branch #2

wants to merge 14 commits into from

Conversation

rjh-24
Copy link
Collaborator

@rjh-24 rjh-24 commented Dec 3, 2024

  • adding variables to config.example that are used within db.php to connect to siteground mysql database
  • adding register.php to handle logic needed for registering new users
  • updating index.php to use connected db for logging in existing users
  • updating ftp_upload to have option for mock api response to save on limited api calls

@rjh-24 rjh-24 requested a review from clearpaint December 3, 2024 19:28
richardjhong and others added 9 commits December 6, 2024 18:10
… tags that meet threshold, updating app.js to account for logic of non-existent tags and fixing downloadImage function to only use tags that are actively checked, updating main.php to show welcome message for logged in username, adding in display_metadata.php for now to test which metadata is able to be retrieved (so far only comments of the embed_metadata.php display)
…dering of checkboxes with labels to have within same horizontal row
…itation of using imagick for writing exif data
…a modal to display metadata for visual confirmation that comments are embedded, adding auto redirect on register to login after 5 seconds, adding functions within app.js to handle modal opening and displaying of metadata, changing when temp_output_path is unlinked to when logging out or when executing php script of display_metadata
… accordingly to config.php, referencing ftp_path where needed in php files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants