Qr codes

closes #49 (closed)

changed milestone to %Week 9 - Sprint #2

added Ready to Merge label

requested review from @d1634883

assigned to @c23102007

changed the description

added 1 commit

• f6d18b80 - QR Updates

Compare with previous version

added 1 commit

• a9767edb - Finished part one

Compare with previous version

added 1 commit

• 68051fc4 - QR part 1

Compare with previous version

added 1 commit

• 347da770 - Added fields into dragonstale database as a test to further integrate QR code...

Compare with previous version

added 1 commit

• 434da3fe - Created a mock user database table for testing.

Compare with previous version

removed Ready to Merge label

added To Do label

added 65 commits

• 434da3fe...f8c5598f - 63 commits from branch main
• 6ebdb676 - Merge remote-tracking branch 'origin/main' into QRCodes
• bedbd7bd - Adding integration into the current Users class to pull dragonstale QR code information out.

Compare with previous version

Thanks for the work, unfortunately there appears to be a couple errors within the code, stopping it from booting up. The rest of your code looks pretty good though.

• error in userRepositoryJDBC line 60, getDTCompletion method
• import 'static Team5.SmartTowns.Landmarks.Landmarks.landmarksDragonstrail' gives an error in Landmarks due to private access
• Within UserRepository, getTest(int id) and getDTCompletion(int id) are unimplemented within UserRepositoryJDBC and cause errors.

added 1 commit

• 8f2bf6a3 - Commented out code for URL query string integration to get a functioning QR...

Compare with previous version

approved this merge request

approved this merge request

merged

mentioned in commit a9ddc9f2

Reviewed and tested in-person with participants. Code works and meets tasks and assessment criteria, I'm happy with your work and have pushed to main.

mentioned in commit e5137b42