Skip to content

Log items that share the same UUID #53

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

Open
Spazcool wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Log items that share the same UUID #53

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
9790b58
REFACTOR: logScanned to catch doubled UUIDs & render doubles on template
Dec 14, 2017
f19fcc8
FIX: hacked out bug
Dec 19, 2017
4aea460
REMOVE: cut duplication in logScanned func
Dec 19, 2017
e4bb3c8
REFACTOR: logScanned works again
Dec 19, 2017
b6dc388
REMOVE: double UUID conditional from recent.ejs
Dec 19, 2017
93da255
ADD: double UUID conditional without duplicate code
Dec 19, 2017
cae791d
ADD: double UUID conditional without duplicate code
Dec 19, 2017
5b37912
ADD: link to item row in spreadsheet to both item obj and recent.ejs
Dec 19, 2017
6f7a97f
Refactoring/simplifying code googleSpreadsheet.js
mpsido Dec 20, 2017
1c1d78a
Removing hard url in app.js
mpsido Dec 20, 2017
d04c2cf
Refactoring, addRecentlyScanned function instead of logScanned
mpsido Dec 20, 2017
9d8e1c8
Merge master
mpsido Dec 29, 2017
12229de
Fixing pre-commit lint warnings, and adding addQrImg function to items
mpsido Dec 29, 2017
3cd5020
moving filterEmptyUuid and sortByFloor functions to attribute of matc…
mpsido Dec 29, 2017
d33e8d9
Removing deprecated similar items function
mpsido Dec 29, 2017
9b0b9a6
Fixing qrList regression
mpsido Dec 29, 2017
e773185
Reintroducing cellRef
mpsido Dec 29, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion app.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ app.get('/qrlist', (req, res) => {
.filter(item => item.uuid !== '')
.filter(item => item.uuid !== undefined)
.sort((a, b) => (a.floor === b.floor ? 0 : +(a.floor > b.floor) || -1));
qrList.forEach(item => item.qr = qr.imageSync('http://url.coderbunker.com/' + item.uuid, { type: 'svg' }));
qrList.forEach(item => item.qr = qr.imageSync(item.uuid, { type: 'svg' }));
res.render('qrList', { matches: qrList });
Copy link
Member

@lenzai lenzai Dec 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

break behaviour , it should be / instead of

Besides, it's off topic with the current branch

Copy link
Contributor

@mpsido mpsido Dec 21, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll fix that, in which case it won't work ? I need to reproduce it so that I ensure the change will fix that case

Copy link
Member

@lenzai lenzai Dec 21, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if current page is not root folder then it will assume the uuid is a subfolder

});
});
Expand All @@ -75,6 +75,9 @@ app.get('/:uuid', (req, res) => {
return;
}
logScanned(req.params.uuid, matches);
if (matches.length >= 1) {
console.log(`Too much matches for uuid ${req.params.uuid} length = ${matches.length}`);
}
matches[0].similarItems = searchDatabase({ fixture: matches[0].fixture }, allItems)
Copy link
Member

@lenzai lenzai Dec 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

off topic

Copy link
Author

@Spazcool Spazcool Dec 20, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps amateur questions, but why remove the hard URL and how else do we get a link to work?

.filter(item => item.uuid !== matches[0].uuid)
.splice(0, 3);
Expand Down