Skip to content

Champs page#68#71

Draft
VincentLeighton wants to merge 13 commits into
mainfrom
ChampsPage#68
Draft

Champs page#68#71
VincentLeighton wants to merge 13 commits into
mainfrom
ChampsPage#68

Conversation

@VincentLeighton

@VincentLeighton VincentLeighton commented Jan 10, 2022

Copy link
Copy Markdown
Collaborator

Description

Progress on the all champs page addition

Screenshot(s) (if applicable, e.g. style updates)

image

PR Readiness Checklist

  • General
    • Verified all changes are related to feature described above (no scope creep)
  • Branch Stuff
    • Checked target and base branch (should be feature branch merging INTO main)
    • Merged latest main into feature branch (so it has latest code)
    • Resolved any conflicts between feature branch and main
  • Test Stuff
    • Added unit tests to cover modified / additional code
  • GitHub Stuff (on the right 👉)
    • Updated labels
    • Linked issue to which this PR relates
    • Added assignee
    • Added at least one (1) reviewer

Added Champion Display Page
Added routing and navigation to Chamption Display Page
Added component for individual champ to be displayed in champ display page
Added interface for ChampDisplay and added IChampDisplay to index
Added API calls to static Dragon page for latest version call and all champion data
@VincentLeighton VincentLeighton added the feature New feature or request label Jan 10, 2022
@VincentLeighton VincentLeighton linked an issue Jan 10, 2022 that may be closed by this pull request
@Wolven531 Wolven531 marked this pull request as draft January 10, 2022 02:54
Completed champion page, added champ images from static png
Updated IChampDisplay and added IChampImage
Updated index
@VincentLeighton

Copy link
Copy Markdown
Collaborator Author

image

Added API call to dragon for single champ data call
Added interfaces for champ info and updated index
Added routing for ChampDisplay page
Added link to ChampInfo page, added ChampInfo page
ChampInfo currently getting champ object but not displaying on page
 Update to API return only champ data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

Development

Successfully merging this pull request may close these issues.

Add page to display champions

2 participants