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

Unpack Components - Non-static Pages #6

Merged
merged 1 commit into from
Jun 11, 2020

Conversation

ahurka
Copy link

@ahurka ahurka commented Jun 8, 2020

Removes all occurrences of the woudc-blurb, woudc-note, woudc-link, and woudc-select components and replaces them with straight-up Vue components in any remaning non-static pages in the project.

Mainly affects list-type pages (e.g. contributors list, stations list) and the search page, as well as the contact page. Does not affect the news page or products/dataset info pages; I'm not clear on how those pages will work just yet.

Intended as part of a cleanup of the translation system, so that blurbs are built using readable component substitution rather than being set up behind-the-scenes by all these custom components. So translations in the JSON files and their uses in the Vue components in non-static pages have been cleaned up too.

@ahurka ahurka force-pushed the another-no-components branch 2 times, most recently from c841420 to f34aba2 Compare June 10, 2020 17:03
Copy link
Contributor

@kngai kngai left a comment

Choose a reason for hiding this comment

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

Added my comments. Mostly they're from the previous PR. Keep in mind about _id.vue pages as they will be needed to be convert to query params. That is, removing _id.vue and putting it into the proper vue page.

locales/en.json Show resolved Hide resolved
pages/contact.vue Show resolved Hide resolved
pages/data/explore.vue Outdated Show resolved Hide resolved
pages/data/explore.vue Outdated Show resolved Hide resolved
ahurka pushed a commit to ahurka/woudc-ui that referenced this pull request Jun 11, 2020
ahurka pushed a commit to ahurka/woudc-ui that referenced this pull request Jun 11, 2020
Take constants out of global scope

Change contributor header translations from lists to objects

Change station header translations from lists to objects

Change instrument header translations from lists to objects

Unpack search page woudc-blurb components

Unpack datasets selector and provide translations

Unpack search page country selector component

Unpack search page station selector component

Unpack search page instrument selector component

Move selector styles into search page

Clean up search page models and add '...' select options

Unpack contact page woudc-blurb components

Unpack contact page woudc-note components

Cleanup contact page models

Replace some placeholder translations

Finalize non-static page routes

Apply pull request woudc#6 suggestions
@kngai kngai merged commit 521a526 into woudc:master Jun 11, 2020
ahurka pushed a commit to ahurka/woudc-ui that referenced this pull request Jun 18, 2020
Take constants out of global scope

Change contributor header translations from lists to objects

Change station header translations from lists to objects

Change instrument header translations from lists to objects

Unpack search page woudc-blurb components

Unpack datasets selector and provide translations

Unpack search page country selector component

Unpack search page station selector component

Unpack search page instrument selector component

Move selector styles into search page

Clean up search page models and add '...' select options

Unpack contact page woudc-blurb components

Unpack contact page woudc-note components

Cleanup contact page models

Replace some placeholder translations

Finalize non-static page routes

Apply pull request woudc#6 suggestions
@ahurka ahurka deleted the another-no-components branch June 22, 2020 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants