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

Community page #2262

Merged
merged 29 commits into from
Dec 12, 2023
Merged

Community page #2262

merged 29 commits into from
Dec 12, 2023

Conversation

AKCodez
Copy link
Contributor

@AKCodez AKCodez commented Nov 14, 2023

No description provided.

Copy link

Link check report. 554131 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

1 similar comment
Copy link

Link check report. 554131 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

Link check report. 554131 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

Link check report. 554131 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

Link check report. 554131 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

Link check report. 554129 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

Link check report. 554129 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

Link check report. 554129 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

Link check report. 554129 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

@intelliot intelliot removed their request for review December 6, 2023 06:01
Copy link

github-actions bot commented Dec 7, 2023

Link check report. 560376 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

1 similar comment
Copy link

github-actions bot commented Dec 7, 2023

Link check report. 560376 links checked.
6 broken links found:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:
File: out/ja/contribute.html Link:

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

@AKCodez AKCodez requested a review from mDuo13 December 7, 2023 14:33
Copy link

github-actions bot commented Dec 7, 2023

Link check report. 560370 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

@AKCodez AKCodez requested a review from tequdev December 7, 2023 14:36
Copy link

github-actions bot commented Dec 7, 2023

Link check report. 560370 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

github-actions bot commented Dec 8, 2023

Link check report. 579172 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

github-actions bot commented Dec 8, 2023

Link check report. 579172 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link
Collaborator

@mDuo13 mDuo13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still a couple fixes that need to go in before we should merge this.

<script type="text/javascript">
document.addEventListener("DOMContentLoaded", function() {
const events = [
{ "name": _("New Horizon: Innovate Without Limits: New Horizons Await"),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The _(...) function for translation doesn't work in this context. You can use it in {% set ... %} blocks which are handled by the Jinja preprocessor (which performs translation at build time) but you can't just use it in client-side JavaScript. The page is throwing an error in the browser console about _ being undefined because of this.

One workaround is to wrap the entire events array here in {{ ... }} which evaluates it as Jinja syntax first, which will do translation and output a dictionary which happens to be compatible with JS syntax… as long as you consistently quote the keys (i.e. the unquoted key literals on lines 291–324 throw an error).

Another problem confounding this is that you have the event details defined in two places and this one is (currently) not being used, the one in contribute-carousel.js is. It's possible to work around that and make it use the translatable one from this file. But you have to do all of the following:

  • remove the events definition from contribute-carousel.js
  • use var instead of const
  • move the script tag for contribute-carousel.js below the one that defines events
  • plus fix the things with translatability I mentioned in the previous paragraph.

Since that's complicated, I've pushed a commit ( 1e6c237 ) that does all of that. But you probably want to look at the updated events list and confirm that it works.

template/page-community.html.jinja Outdated Show resolved Hide resolved
@mDuo13 mDuo13 added needs japanese translation PRs with English changes that need to be added/updated in the Japanese translation of the site web dev Styles, templates, or other technologies for how the site is built & presented labels Dec 8, 2023
Copy link

github-actions bot commented Dec 8, 2023

Link check report. 579172 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Co-authored-by: Rome Reginelli <rome@ripple.com>
Copy link

github-actions bot commented Dec 9, 2023

Link check report. 579172 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

Copy link

Link check report. 579093 links checked.
Success! No broken links found.

Preview: https://XRPLF.github.io/xrpl-dev-portal/pr-preview/community-page/

Style Report

@mDuo13 mDuo13 merged commit 1d570e3 into master Dec 12, 2023
3 checks passed
@mDuo13 mDuo13 deleted the community-page branch December 12, 2023 01:37
@tequdev tequdev removed the needs japanese translation PRs with English changes that need to be added/updated in the Japanese translation of the site label Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
web dev Styles, templates, or other technologies for how the site is built & presented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants