Skip to content
This repository has been archived by the owner on May 24, 2023. It is now read-only.

New Product Review views & checkout card #46

Merged
merged 21 commits into from
Jul 11, 2016
Merged

Conversation

GeroHerkenrath
Copy link
Contributor

@GeroHerkenrath GeroHerkenrath commented Jul 11, 2016

Implementing Jira Tickets APP-306 and APP-299 at once.
Now we have product review views and the card shown during checkout adapts its dimensions (mostly its height, the width still needs to be fixed in the actual code that generates the card, i.e. in the backend)

Gero Herkenrath added 21 commits July 4, 2016 18:07
created an abstract (more or less) base view class that can be used to more easily subclass views suited for our loading mechanisms
started a private internal base view (non abstract) for product grades.
added necessary category for TRSNetworkAgent.
refactored shop grade stuff into own category for TRSNetworkAgent.
refactored tests for the latter thing.
added commons category to TRSNetworkAgent (error handling etc.)
added missing UI orientation to example project (this sometimes threw a warning, but not always... weird)
split headers into private and (soon to be public) parts
added error checking & handling basic data view
moved some documentation into private headers
implemented SKU hashing function and finished ProductGrade category for TRSNetworkAgent
updated example project to show view
alignment and example for 1 line mode is missing
updated tests for smaller possible stars
minor cleanup in comments
also included creation or take over dates in headers of test (so one can see who is responsible)
also found and fixed a mocking mistake I made in private basic data view
re-added the deprecated error code symbols and marked them accordingly
added better appledoc way (well... "better") to mark deprecated error symbols.
…is bigger than the default lightbox)

added tests for that
prepared readme and changelog for release
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 60a1cfb on feature/APP-306 into f26e427 on master.

@GeroHerkenrath GeroHerkenrath merged commit f130ef7 into master Jul 11, 2016
@GeroHerkenrath GeroHerkenrath deleted the feature/APP-306 branch July 11, 2016 11:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants