Latest revision |
Your text |
Line 318: |
Line 318: |
| | | |
| - Admin Meteor User Creation Issue https://github.com/BRL-CAD/OGV-meteor/pull/91/commits/06425846802069420e0fe5da42b143e9922d2b4a | | - 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.
| |