Skip to content
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.

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

Patreon Integration #6

Closed
3 tasks done
ghost opened this issue Jan 27, 2021 · 6 comments · Fixed by #48
Closed
3 tasks done

Patreon Integration #6

ghost opened this issue Jan 27, 2021 · 6 comments · Fixed by #48
Assignees
Labels
Blocked When work cannot process until another issue is resolved feature New feature or request

Comments

@ghost
Copy link

ghost commented Jan 27, 2021

With the Patrons being on the home page, they aren't also needed in the footer.

  • Remove footer

Integrate the news and patrons

  • Add "news" from Patreon
  • Add "Patron list" from Patreon

Patreon API: https://ultsky.phinocio.com/api/patreon

interface Patron {
    name: string
    tier: "Patron" | "Super Patron"
}

interface Post {
    title: string
    content: string
    published: string
    url: string
}

interface PatreonsResponse {
    patrons: Patron[]
    posts: Post[]
}

@ghost ghost assigned phinocio and thomasblasquez Jan 27, 2021
@ghost ghost added the feature New feature or request label Jan 27, 2021
@ghost ghost self-assigned this Jan 27, 2021
@MattLish MattLish self-assigned this Mar 10, 2021
@MattLish MattLish added this to the V1.0.0 milestone Mar 10, 2021
@MattLish MattLish unassigned ghost and thomasblasquez Mar 10, 2021
@MattLish
Copy link
Member

Progress for adding Patreon list tracked in https://github.com/UltimateSkyrim/ultskylauncher/tree/feat/patreon-list

@MattLish
Copy link
Member

Patreon list has been added with #44

@MattLish
Copy link
Member

Progress tracked here https://github.com/UltimateSkyrim/ultskylauncher/tree/feat/news-list

Blocked by #46 because the p tag is used in news and this style is globally set.

@MattLish MattLish added the Blocked When work cannot process until another issue is resolved label Mar 24, 2021
@MattLish
Copy link
Member

I was incorrectly trying to render the content instead of the title. Titles shouldn't contain HTML so this is no longer blocked.

@MattLish MattLish removed the Blocked When work cannot process until another issue is resolved label Mar 24, 2021
@MattLish
Copy link
Member

MattLish commented Mar 24, 2021

Outstanding work:

  • Open links in a new browser window (Electron doesn't handle this by default)

  • Apply colours to tags (news and devstream should be coloured as per the design and other tags should be grey)

  • Fix issue where the date bleeds into the background
    image

  • Fix misalignment of tags
    image

@MattLish
Copy link
Member

Blocked by #47

@MattLish MattLish added the Blocked When work cannot process until another issue is resolved label Mar 25, 2021
MattLish added a commit that referenced this issue Mar 25, 2021
- Add ExternalLink component
- Add posts service

closes #6
MattLish added a commit that referenced this issue Mar 30, 2021
- Add ExternalLink component
- Add posts service

closes #6
MattLish added a commit that referenced this issue Mar 30, 2021
- Add ExternalLink component
- Add posts service

closes #6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked When work cannot process until another issue is resolved feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants