Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don`t duplicate repo data if two users have GitHub access to the same repo #7

Open
alexmchp opened this issue Mar 17, 2020 · 0 comments
Open

Don`t duplicate repo data if two users have GitHub access to the same repo #7

alexmchp opened this issue Mar 17, 2020 · 0 comments
Labels
optimization Helps save some resource

Comments

@alexmchp
Copy link
Collaborator

Currently if two users have GitHub "native" access to a repo (for example both are admins) the views data for that repo are saved twice (once for each user). Possible optimization is that the data are saved only once. This means that in the repo model instead of having a single user there must be an array of users.
@alexmchp alexmchp added the optimization Helps save some resource label Mar 17, 2020
alexmchp added a commit that referenced this issue Oct 4, 2021
@alexmchp
Merge pull request #7 in M8A/github-views from develop to master
6846604 
* commit 'c6b6de242769bf4d65d3127f3110867749bdae4f': (38 commits)
  Added forks delta in migration script; Removed unused code from index.js; Fixed forks delta in cron-job.
  Removed backup 2020-08-24; moved start_fresh script into scripts folder.
  Prepare forks series in Data context. Store forks as delta.
  Promise based updaete; store new repos in object, before saving them at the end of processing, to avoid iteration steps entanglement.
  Debug.
  New implementation for database migration.
  Added variable undefine test in filter function.
  Bug fix. Adding the the created repository in de local array.
  Replaced undefined variable.
  Undefined variable replaced with the correct name.
  When a new repo is created, it should be added to the repo list, so next searches in the repos array will find it.
  Store delta for forks.
  Fiex errors after merge. Fixed Excel sheet tile rows too long.
  Created updateRepoCommits function to update a repository commits in database.
  Added commits data to the new created repository.
  Added getRepoCommits function.
  Commits can be updated only one time per day. In cronjob the update variable will be resetted to false value.
  Added commits in database Repository Schema.
  Moved file download function into utils.js
  Implemented generator run
  ...
dependabot bot added a commit that referenced this issue Oct 4, 2021
@dependabot
Merge pull request #7 in M8A/github-views from develop to master
7298a7d 
* commit 'c6b6de242769bf4d65d3127f3110867749bdae4f': (38 commits)
  Added forks delta in migration script; Removed unused code from index.js; Fixed forks delta in cron-job.
  Removed backup 2020-08-24; moved start_fresh script into scripts folder.
  Prepare forks series in Data context. Store forks as delta.
  Promise based updaete; store new repos in object, before saving them at the end of processing, to avoid iteration steps entanglement.
  Debug.
  New implementation for database migration.
  Added variable undefine test in filter function.
  Bug fix. Adding the the created repository in de local array.
  Replaced undefined variable.
  Undefined variable replaced with the correct name.
  When a new repo is created, it should be added to the repo list, so next searches in the repos array will find it.
  Store delta for forks.
  Fiex errors after merge. Fixed Excel sheet tile rows too long.
  Created updateRepoCommits function to update a repository commits in database.
  Added commits data to the new created repository.
  Added getRepoCommits function.
  Commits can be updated only one time per day. In cronjob the update variable will be resetted to false value.
  Added commits in database Repository Schema.
  Moved file download function into utils.js
  Implemented generator run
  ...
dependabot bot added a commit that referenced this issue Oct 4, 2021
@dependabot
Merge pull request #7 in M8A/github-views from develop to master
4236615 
* commit 'c6b6de242769bf4d65d3127f3110867749bdae4f': (38 commits)
  Added forks delta in migration script; Removed unused code from index.js; Fixed forks delta in cron-job.
  Removed backup 2020-08-24; moved start_fresh script into scripts folder.
  Prepare forks series in Data context. Store forks as delta.
  Promise based updaete; store new repos in object, before saving them at the end of processing, to avoid iteration steps entanglement.
  Debug.
  New implementation for database migration.
  Added variable undefine test in filter function.
  Bug fix. Adding the the created repository in de local array.
  Replaced undefined variable.
  Undefined variable replaced with the correct name.
  When a new repo is created, it should be added to the repo list, so next searches in the repos array will find it.
  Store delta for forks.
  Fiex errors after merge. Fixed Excel sheet tile rows too long.
  Created updateRepoCommits function to update a repository commits in database.
  Added commits data to the new created repository.
  Added getRepoCommits function.
  Commits can be updated only one time per day. In cronjob the update variable will be resetted to false value.
  Added commits in database Repository Schema.
  Moved file download function into utils.js
  Implemented generator run
  ...
dependabot bot added a commit that referenced this issue Oct 4, 2021
@dependabot
Merge pull request #7 in M8A/github-views from develop to master
63841c6 
* commit 'c6b6de242769bf4d65d3127f3110867749bdae4f': (38 commits)
  Added forks delta in migration script; Removed unused code from index.js; Fixed forks delta in cron-job.
  Removed backup 2020-08-24; moved start_fresh script into scripts folder.
  Prepare forks series in Data context. Store forks as delta.
  Promise based updaete; store new repos in object, before saving them at the end of processing, to avoid iteration steps entanglement.
  Debug.
  New implementation for database migration.
  Added variable undefine test in filter function.
  Bug fix. Adding the the created repository in de local array.
  Replaced undefined variable.
  Undefined variable replaced with the correct name.
  When a new repo is created, it should be added to the repo list, so next searches in the repos array will find it.
  Store delta for forks.
  Fiex errors after merge. Fixed Excel sheet tile rows too long.
  Created updateRepoCommits function to update a repository commits in database.
  Added commits data to the new created repository.
  Added getRepoCommits function.
  Commits can be updated only one time per day. In cronjob the update variable will be resetted to false value.
  Added commits in database Repository Schema.
  Moved file download function into utils.js
  Implemented generator run
  ...
dependabot bot added a commit that referenced this issue Oct 4, 2021
@dependabot
Merge pull request #7 in M8A/github-views from develop to master
d8b9f22 
* commit 'c6b6de242769bf4d65d3127f3110867749bdae4f': (38 commits)
  Added forks delta in migration script; Removed unused code from index.js; Fixed forks delta in cron-job.
  Removed backup 2020-08-24; moved start_fresh script into scripts folder.
  Prepare forks series in Data context. Store forks as delta.
  Promise based updaete; store new repos in object, before saving them at the end of processing, to avoid iteration steps entanglement.
  Debug.
  New implementation for database migration.
  Added variable undefine test in filter function.
  Bug fix. Adding the the created repository in de local array.
  Replaced undefined variable.
  Undefined variable replaced with the correct name.
  When a new repo is created, it should be added to the repo list, so next searches in the repos array will find it.
  Store delta for forks.
  Fiex errors after merge. Fixed Excel sheet tile rows too long.
  Created updateRepoCommits function to update a repository commits in database.
  Added commits data to the new created repository.
  Added getRepoCommits function.
  Commits can be updated only one time per day. In cronjob the update variable will be resetted to false value.
  Added commits in database Repository Schema.
  Moved file download function into utils.js
  Implemented generator run
  ...
dependabot bot added a commit that referenced this issue Oct 4, 2021
@dependabot
Merge pull request #7 in M8A/github-views from develop to master
b400b83 
* commit 'c6b6de242769bf4d65d3127f3110867749bdae4f': (38 commits)
  Added forks delta in migration script; Removed unused code from index.js; Fixed forks delta in cron-job.
  Removed backup 2020-08-24; moved start_fresh script into scripts folder.
  Prepare forks series in Data context. Store forks as delta.
  Promise based updaete; store new repos in object, before saving them at the end of processing, to avoid iteration steps entanglement.
  Debug.
  New implementation for database migration.
  Added variable undefine test in filter function.
  Bug fix. Adding the the created repository in de local array.
  Replaced undefined variable.
  Undefined variable replaced with the correct name.
  When a new repo is created, it should be added to the repo list, so next searches in the repos array will find it.
  Store delta for forks.
  Fiex errors after merge. Fixed Excel sheet tile rows too long.
  Created updateRepoCommits function to update a repository commits in database.
  Added commits data to the new created repository.
  Added getRepoCommits function.
  Commits can be updated only one time per day. In cronjob the update variable will be resetted to false value.
  Added commits in database Repository Schema.
  Moved file download function into utils.js
  Implemented generator run
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
optimization Helps save some resource
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@alexmchp