Editing User:Quentinpaden

From BRL-CAD

Warning: You are not logged in. Your IP address will be publicly visible if you make any edits. If you log in or create an account, your edits will be attributed to your username, along with other benefits.

The edit can be undone. Please check the comparison below to verify that this is what you want to do, and then save the changes below to finish undoing the edit.
Latest revision Your text
Line 287: Line 287:
  
 
==July 27==
 
==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
+
* Learning more about the mail server rest api in meteor and https://dev.mailjet.com/ 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==
 
==July 28==
Line 308: Line 308:
  
 
* 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.
 
* 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.
 

Please note that all contributions to BRL-CAD may be edited, altered, or removed by other contributors. If you do not want your writing to be edited mercilessly, then do not submit it here.
You are also promising us that you wrote this yourself, or copied it from a public domain or similar free resource (see BRL-CAD:Copyrights for details). Do not submit copyrighted work without permission!

To edit this page, please answer the question that appears below (more info):

Cancel Editing help (opens in new window)