| Latest revision |
Your text |
| Line 273: |
Line 273: |
| | ==July 22== | | ==July 22== |
| | * Looking into exactly where the user signs up in accounts.js and putting an logic statement there to check for the user admin email server to avoid the email error | | * Looking into exactly where the user signs up in accounts.js and putting an logic statement there to check for the user admin email server to avoid the email error |
| − |
| |
| − | ==July 23==
| |
| − | * Enhancing code quality of solution provided for Meteor User Creation with multiple new comments and proper formatting in https://github.com/BRL-CAD/OGV-meteor/pull/91
| |
| − |
| |
| − | ==July 24==
| |
| − | * Looking into exactly where the user signs up in accounts.js and properly commenting the code to understand a solution for the email server error in https://github.com/BRL-CAD/OGV-meteor/issues/57
| |
| − |
| |
| − | ==July 25==
| |
| − | * Looking into fixing https://github.com/BRL-CAD/OGV-meteor/issues/57 with the email server error through checking if the admin SMTP settings are properly set in the Online Geometry Viewers server
| |
| − |
| |
| − | ==July 26==
| |
| − | * Looking into exactly how to get the email server information and preventing the sign up/login function untill the email server information has been retrieved to check the required settings variable more dynamically and prevent the error
| |
| − |
| |
| − | ==July 27==
| |
| − | * Learning more about the mail server rest api in meteor and Mailgun email server to learn more about the SMTP settings and how it's set on the server, finding which api is providing the mail function
| |
| − |
| |
| − | ==July 28==
| |
| − | * Today is my birthday! Getting a lot closer to learning about a solution for https://github.com/BRL-CAD/OGV-meteor/issues/57 through
| |
| − |
| |
| − | ==July 29==
| |
| − | * Moving out back to school. Will be researching https://docs.meteor.com/api/email.html for the vanilla meteor email api without plugins
| |
| − |
| |
| − | ==July 30==
| |
| − | * Moving out back to school. Looking through Github code for https://github.com/BRL-CAD/OGV-meteor/issues/57 and finding the signin/login function api for https://docs.meteor.com/api/accounts.html#Meteor-loginWithPassword in relation to the sign in and seeing if I can find all instances the proper sign in function with the meteor server to properly add a conditional statement to them
| |
| − |
| |
| − | ==July 31==
| |
| − | * Arrived at the move in destination. Am unpacking and temporarily without internet to test the meteor server, looking to see if I can test it offline
| |
| − |
| |
| − | ==August 1==
| |
| − | * Internet is back online. Updated Dev Log. Submitting another PR to look further into https://github.com/BRL-CAD/OGV-meteor/issues/57 . Submitting code quality updates to close https://github.com/BRL-CAD/OGV-meteor/pull/91
| |
| − |
| |
| − | * Formatted a Pull Request for solving the Admin Email Server Error Issue
| |
| − | https://github.com/BRL-CAD/OGV-meteor/pull/89/files#diff-85207ec3ffd02327adfb13aec22778a2
| |
| − |
| |
| − | * In submitting this PR to fix the Admin Email Server Issue, I added a required settings function (isAdminEmailServerOn) to validator.js, doing this will allow me to check Login.js and Signin.js if the required meteor SMTP email settings have been saved before allowing Logins and Signups into the meteor server for OGV. https://github.com/BRL-CAD/OGV-meteor/issues/57.
| |
| − |
| |
| − | ==August 2nd==
| |
| − | * Looking further into reconnecting with Mailgun email server in https://github.com/BRL-CAD/OGV-meteor/pull/89/commits/6b62bc7e602aa6b0ffece8f94d2a2c4950d1881c#diff-154cbb884a1c6d0af4d870bac36cb31aR70 with more information about their api with REST and METEOR from https://documentation.mailgun.com/en/latest/user_manual.html
| |
| − |
| |
| − | ==August 3rd==
| |
| − | * Fixed previous Pull Request for code quality and syntax accuracy:
| |
| − |
| |
| − | - Admin Email Server Error PR https://github.com/BRL-CAD/OGV-meteor/pull/89/commits/6b62bc7e602aa6b0ffece8f94d2a2c4950d1881c
| |
| − |
| |
| − | - Admin Meteor User Creation Issue https://github.com/BRL-CAD/OGV-meteor/pull/91/commits/06425846802069420e0fe5da42b143e9922d2b4a
| |
| − |
| |
| − | ==August 4th==
| |
| − | * reviewing code review from https://github.com/BRL-CAD/OGV-meteor/pull/91#issuecomment-518092734 and looking testing the new solution with meteor reset
| |
| − |
| |
| − | ==August 5th==
| |
| − | * reviewing code review from https://github.com/BRL-CAD/OGV-meteor/pull/91#issuecomment-518092734 making sure admin password is set in meteor settings
| |
| − |
| |
| − | ==August 6th==
| |
| − | * reviewing code review from https://github.com/BRL-CAD/OGV-meteor/pull/91#issuecomment-518092734 confirming the password is set properly and the solution is working
| |
| − |
| |
| − |
| |
| − | ==August 7th==
| |
| − | * Submitting a PR commit to close this issue https://github.com/BRL-CAD/OGV-meteor/pull/91/commits/9b28a0d261fff70cf2257b0ce3b6bb7d19f4c74b
| |
| − |
| |
| − |
| |
| − | ==August 8th==
| |
| − | * Finding an open CDN to load a loading bar asynchronously to minimize and eliminate the extra loading.js package bundle download for CSS for loading bars https://github.com/BRL-CAD/OGV-meteor/issues/62
| |
| − |
| |
| − | ==August 9th==
| |
| − | * Researching in the code exactly where an upload to the server is described to show an alert if an upload to the server fails for https://github.com/BRL-CAD/OGV-meteor/issues/63
| |
| − |
| |
| − | ==August 10th==
| |
| − | * Looking into THREE.js to learn how to accurately get the auto width/heigh ratio for https://github.com/BRL-CAD/OGV-meteor/issues/66
| |
| − |
| |
| − | ==August 11th==
| |
| − | * Submitting a PR commit for settings in Meteor.settings the new adminPassword https://github.com/BRL-CAD/OGV-meteor/pull/91/commits/61f7392eb02d72fbc59349d976d14716f78a257f
| |
| − |
| |
| − | ==August 12th==
| |
| − | * Learning from code reviews from the previous commit. Looking into removing the unwanted files from the email server commit. This PR should only contain validator.js, signup.js, and login.js, but it has files from the PR before it got renamed which I must look into removing. This may not be possible without deleting and readding the files from the branches, closing the PR and creating a new one, or matching the commits on the files to be the same as the other PR so it does not conflict
| |
| − |
| |
| − | ==August 13th==
| |
| − | * Going through trial and error with the command line interface for git and github. Having issues with redacting a file on github with git CLI, researching into merging local and remote branches with git CLI instead. May install a plugin for VSCODE with a different github CLI to more easily complete this operation.
| |
| − |
| |
| − | ==August 14th==
| |
| − | * Going through trial and error learning and using the VSCODE command line interface for github to merge the branches on the PR's, is giving errors and warnings about the changes deleting the previous commits. Trying to solve both PR's without deleting any files. May have to just match the files in both PR's to avoid conflict if merging the branches locally and remotely does not work.
| |
| − |
| |
| − | ==August 15th==
| |
| − | * Before matching the branches, working on implementing a more final solution per the new review given on https://github.com/BRL-CAD/OGV-meteor/pull/91. Looking into making sure this solution works every time the meteor server runs.
| |
| − |
| |
| − | ==August 16th==
| |
| − | * Implemented the solution after testing, now making sure the branches from patch-1 and development match before commiting to both Pull Requests to avoid conflicts
| |
| − |
| |
| − | ==August 17th==
| |
| − | * Resetting Git Locally on computer after dealing with local file path errors when trying to commit. Uninstalling the VSCODE plugin CLI for Github, and installing the official new Microsoft Github CLI for git and not the vanilla git cli
| |
| − |
| |
| − | ==August 18th==
| |
| − | * Taking a break for 2 days to be with sister after her surgery for lung cancer.
| |
| − |
| |
| − | ==August 21th==
| |
| − | * Logic change for account.js for Meteor Admin User Password Pull Request
| |
| − | * https://github.com/BRL-CAD/OGV-meteor/pull/91/commits/8a2748cbf7f5b472f6d472a175daa97760d34437
| |
| − |
| |
| − | * Matching Logic change commit with account.js on the other PR on branch patch-1 to avoid any conflicts with merging and closing both pull requests.
| |
