Skip to content

Ability to Sort Newsletter Archives #4058

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
Closed
kkartunov opened this issue Mar 3, 2020 · 5 comments
Closed

Ability to Sort Newsletter Archives #4058

kkartunov opened this issue Mar 3, 2020 · 5 comments
Assignees
@kkartunov
Labels
Contentful-DEV Internal development operations for Contentful Prod Env Environment QA Pass
Milestone
v0.32.6

Comments

@kkartunov
Copy link
Collaborator

Update newsletter archives component to match new design https://marvelapp.com/535c9ga/screen/66441678/handoff
@kkartunov kkartunov self-assigned this Mar 3, 2020
@kkartunov kkartunov added the Contentful-DEV Internal development operations for Contentful label Mar 3, 2020
kkartunov added a commit that referenced this issue Apr 30, 2020
@kkartunov
Implements #4058
2fd6475
@kkartunov kkartunov mentioned this issue Apr 30, 2020
Merged
@SathyaJayabal SathyaJayabal added this to the current_contentful milestone Apr 30, 2020
@Jobee1
Copy link
Collaborator

Jobee1 commented May 1, 2020

There is issue in this one.
In the design of the newsletter, there is "Load More" button to load the number of articles but in the test environment, there is scroll bar to scroll the number of articles.

Attached screenshot for the reference:
As mentioned in the design:
Screenshot_38

As displayed in Test Environment:
Screenshot_39

@Jobee1 Jobee1 added QA Fail QA verification on Dev has failed. Assignee to redo the fix. Test Env Environment labels May 1, 2020
@kkartunov
Copy link
Collaborator Author

@Jobee1 @SathyaJayabal There won't be "Load More" button. It is out of scope as we decided to load all data in one call and skip load more functionality.

@kkartunov kkartunov added Ready for QA and removed QA Fail QA verification on Dev has failed. Assignee to redo the fix. labels May 1, 2020
@Jobee1
Copy link
Collaborator

Jobee1 commented May 1, 2020 •
edited
Loading

Verified in Test environment on the basis of the comment.
Attached video for reference.
4058.zip

@SathyaJayabal
Copy link
Collaborator

verified on beta(develop branch)
Screenshot 2020-05-20 at 12 15 10 PM

@SathyaJayabal SathyaJayabal added Beta Env Environment and removed Test Env Environment labels May 20, 2020
@kkartunov kkartunov mentioned this issue May 21, 2020
Merged
@SathyaJayabal SathyaJayabal added Prod Env Environment and removed Beta Env Environment labels May 21, 2020
@SathyaJayabal
Copy link
Collaborator

verified on prod
Screenshot 2020-05-21 at 4 25 33 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kkartunov kkartunov

Labels
Contentful-DEV Internal development operations for Contentful Prod Env Environment QA Pass
Projects
None yet
Milestone
v0.32.6
Development

No branches or pull requests
3 participants
@kkartunov @Jobee1 @SathyaJayabal