profile
viewpoint
Rudo Kemper rudokemper Amazon Conservation Team South America http://terrastories.io Geographer working to help make maps and build tech for native communities.

rudokemper/Open-Governance 0

Governing and organizing repository for Tech Talk DC

rudokemper/rudokemper.github.io 0

Agency Theme for Jekyll

rudokemper/storytelling 0

Storytelling with maps template

rudokemper/terrastories-minimal 0

A stripped down version of the Terrastories application which runs just the tilebuilder and tileserver.

rudokemper/terrastories-ulupuene 0

Terrastories is a geostorytelling application built to enable local communities to locate and map their own oral storytelling traditions about places of significant meaning or value to them.

issue commentTerrastories/terrastories

[UI] Add full size / enlarged image mode for Place photos

Just indexing that Digital Democracy have a great solution for this for their Mapeo Web Viewer app. It enlarges not only the image, but the popup as a whole which is very nice. When their team shares the relevant repo with me, I'll share that here so we can refer to that in building out this feature.

mirandawang

comment created time in 2 hours

issue commentTerrastories/terrastories

Add Projects and links to docs

I would suggest we keep using Projects to group together related issues. I just updated the bug squashing Project to be able to link to all of the bug tickets in the Wiki Roadmap.

cc @FeminismIsAwesome

kalimar

comment created time in 2 hours

GollumEvent
GollumEvent
GollumEvent
GollumEvent
GollumEvent
GollumEvent

issue closedTerrastories/terrastories

[Video] Add ability to embed YouTube, Vimeo and videos hosted elsewhere

It is possible to upload videos directly to Terrastories, but for any online instances of the application, it would be nice to be able to just embed videos already on YouTube or Vimeo to the Stories.

closed time in 3 hours

rudokemper
GollumEvent
GollumEvent

issue commentTerrastories/terrastories

CSV importer fails for rows without a media attachment

I can confirm that the same issue arises for Stories without a media attachment.

image

rudokemper

comment created time in 14 days

issue commentTerrastories/terrastories

CSV importer fails for rows without a media attachment

This is what the CSV file I'm using looks like. The importer fails and gives the above message when it comes across the last row in this screenshot, Afitimama

image

rudokemper

comment created time in 14 days

issue openedTerrastories/terrastories

CSV importer fails for rows without a media attachment

The CSV importer seems to fail whenever it comes across a row that does not have a media attachment. See screenshot below.

This issue came up when importing Places using the importer. All of the Places imported without a problem until the importer came across a row for a Place without a media attachment. Then I got the following message.

If there is no media attachment for a Place (or other object), the CSV importer should just move on to the next row, without attaching anything.

image

created time in 14 days

issue commentTerrastories/terrastories

Scroll bar not rendering properly in Chrome (Windows)

I just checked this. So, I can confirm the issue on Chrome on my Surface Pro 7 (Windows 10 Home) using the Heroku App version.

But, I don't see it on Chrome on a Dell Inspiron 15 7000 (Windows 10 Pro).

Same version of Chrome (Version 83.0.4103.116).

rudokemper

comment created time in 15 days

issue commentTerrastories/terrastories

Changes to marker popup layout and content

Fair point -- sorry if I wasn't clear with these issues. I was thinking that for this issue #472, we're just worrying about the photo and the description and essentially leaving the metadata fields alone, but I see that we're doing some bolding here to enhance the legibility of the metadata fields.

I was thinking of issue #473 as the next step, treating the metadata by displaying the information in a sentence format.

As for the legibility, that's an interesting discussion in the tech for indigenous communities space. We tend to veer towards using full sentences as often as possible, because some indigenous peoples, particularly elders, are not used to seeing information presented in that format. But let me give it some thought, because you're right that some communities might prefer the format above. Perhaps we could provide both options? I'll close to issue for now.

rudokemper

comment created time in 15 days

issue closedTerrastories/terrastories

Full sentences in i18n for Place and Story metadata (English)

Currently, for the Place model metadata information rendered in the place marker popup, we are showing this as:

Type of place: [type_of_place] Region: [region]

Let's improve on this by providing this information using full sentences, using the en.yml i18n file. The sentence could read: [Place] is a [type of place] in the [region] region.

Similarly, we should add i18n translation for the Story model metadata, currently not being shown in the story cards.

This conversation mentions the following places: [place]. The conversation happened on [date] at [interview location]. The interviewer is [interviewer]. The language of the conversation is [language].

Once we have the English i18n in place, we can work to provide translations for the other supported languages.

closed time in 15 days

rudokemper

issue openedTerrastories/terrastories

Marker popup doesn't close when the marker disappears from the map

When you click on a marker, a popup opens. But when you make that marker disappears i.e. by adjusting the filters, the popup stays open.

image

Let's make it so that the popup closes automatically whenever the marker disappears from the map.

This issue can be observed on the Heroku demo.

created time in 15 days

issue commentTerrastories/terrastories

Changes to marker popup layout and content

Hi @ohamuy, looks great! I favor the second version. Can we also have slightly more top and bottom padding on the description text, let's say 5px? Thanks so much!

rudokemper

comment created time in 15 days

issue commentTerrastories/terrastories

As a teacher logged in to terrastories, I can add stories to a given curriculum

@FeminismIsAwesome @slaloggia I was able to add stories to the curriculum!

I only wonder about the mechanism that we use to do so, and the UI. Right now it's an autotag list (there probably is a more technical term), but it would be nice for them to show them in a table format with more information about the stories beside the name, as per the wireframes above.

But I might be getting ahead of myself. If the purpose of this ticket is just to be able to add stories to the curriculum and not worry about the UI just yet, this is great 👍

FeminismIsAwesome

comment created time in 19 days

issue commentTerrastories/terrastories

As a teacher logged in to Terrastories, I can create a Curriculum

@FeminismIsAwesome @slaloggia I was able to do exactly what this ticket asks for! 👍 LGTM

FeminismIsAwesome

comment created time in 19 days

issue openedTerrastories/terrastories

Full sentences in i18n for Place and Story metadata (English)

Currently, for the Place model metadata information rendered in the place marker popup, we are showing this as:

Type of place: [type_of_place] Region: [region]

Let's improve on this by providing this information using full sentences, using the en.yml i18n file. The sentence could read: [Place] is a [type of place] in the [region] region.

Similarly, we should add i18n translation for the Story model metadata, currently not being shown in the story cards.

This conversation mentions the following places: [place]. The conversation happened on [date] at [interview location]. The interviewer is [interviewer]. The language of the conversation is [language].

Once we have the English i18n in place, we can work to provide translations for the other supported languages.

created time in 21 days

issue openedTerrastories/terrastories

Changes to marker popup layout and content

Currently, the marker popups are showing only the Region and Type of place fields. We should have Description included from the Place model as well, as well and lay it all out in the following fashion:

Image (100% width as per Oliver's PR already in submission)

Description (with some top and bottom padding to create space between the images and the other two text fields)

Type of place (text retrieved from i18n): type of place Region (text retrieved from i18n): region

created time in 21 days

startedExCiteS/Sapelli

started time in 22 days

issue commentTerrastories/terrastories

Minor adjustment to image rendering in the marker pop ups

@ohamuy assigned!

rudokemper

comment created time in 22 days

issue openedTerrastories/terrastories

Scroll bar not rendering properly in Chrome (Windows)

In Chrome, the sidebar scroll bar is not rendering as it should. It doesn't adapt to the full height of the overflow content.

image

By contrast, it does render properly on Firefox, same computer.

image

This issue has been observed on various Windows machines.

created time in 22 days

issue commentTerrastories/terrastories

[UI] Card height not properly adjusting to content

May be of relevance: https://stackoverflow.com/questions/43837279/dynamic-row-heights-with-react-virtualized-and-new-cellmeasurer

rudokemper

comment created time in 22 days

IssuesEvent

issue openedTerrastories/terrastories

Minor adjustment to image rendering in the marker pop ups

Let's make the photo fill the full width of the pop up, instead of floating to the left of the text fields (as in the below screenshot). The text can go underneath the image. Let's also ensure that image proportions are respected, not shrunk as below.

image

Related follow up issue: https://github.com/Terrastories/terrastories/issues/350

created time in 22 days

issue closedTerrastories/terrastories

Let's add some more sample seed data to Terrastories

Let's add more stories and places to the Terrastories sample seed data, so we can better show off the core front end features (map, filters, etc.) of the app.

closed time in 22 days

rudokemper

create barnchTerrastories/terrastories

branch : 468-add-seed-data

created branch time in 22 days

issue openedTerrastories/terrastories

Let's add some more sample seed data to Terrastories

Let's add more stories and places to the Terrastories sample seed data, so we can better show off the core front end features (map, filters, etc.) of the app.

created time in 22 days

PR closed Terrastories/terrastories

Reviewers
Update SETUP.md

Added missing command for Windows for setup.

Addresses this issue: https://github.com/Terrastories/terrastories/issues/438

+5 -1

1 comment

1 changed file

rudokemper

pr closed time in a month

issue closedTerrastories/terrastories

SETUP.md readme is missing one additional step for Windows

The command docker-compose exec web bin/setup does not work in Windows if you don't run the following command first: docker exec terrastories_web_1 /bin/bash.

closed time in a month

rudokemper

issue commentTerrastories/terrastories

SETUP.md readme is missing one additional step for Windows

I can confirm that it's (no longer?) necessary working with a fresh build of master. Closing this issue and PR.

rudokemper

comment created time in a month

issue commentTerrastories/terrastories

SETUP.md readme is missing one additional step for Windows

@lauramosher or @kathenry -- did you run into this issues on your Windows builds of Terrastories as well? I'll try to install the latest build of Terrastories fresh on my Windows machine tomorrow, and if we don't need to input docker exec terrastories_web_1 /bin/bash anymore, I'll close this issue and withdraw my PR.

(cc @FeminismIsAwesome)

rudokemper

comment created time in a month

delete branch Terrastories/terrastories

delete branch : 408-add-padding-to-fitBounds-2

delete time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 35ffbf93218fe5661843ae7a291fd6de489600e9

Update Map.jsx with padding for fitBounds (#430)

view details

push time in a month

PR merged Terrastories/terrastories

Reviewers
Update Map.jsx with padding for fitBounds

Addresses #408.

Found the padding parameter in the Mapbox GL JS documentation and applied it. 50 pixels seems like an appropriate distance. I tried other pixel sizes to confirm that it's adjusting appropriately; it is.

image

+1 -1

1 comment

1 changed file

rudokemper

pr closed time in a month

issue closedTerrastories/terrastories

[Map] Adjust map maximum zoom extent when using filters

When using the filters, Terrastories zooms in/out to the maximum extent of the places on the map.

Currently, it zooms in/out to the exact coordinates of the places that are the most North/East/South/West in the map view, with the result that these places appear at the borders of the browser window.

Let's make it so that the zooming to maximum extent leave a bit more space in the margins, so that the markers of these furthermost places are entirely contained within the browser window.

closed time in a month

rudokemper

delete branch Terrastories/terrastories

delete branch : 259-423-remove-resetmaptocenter-trigger-2

delete time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha e6eac0a691ee111e43741f6da06aaba7174e8e00

Update Map.jsx removing resetMapToCenter() (#431)

view details

push time in a month

PR merged Terrastories/terrastories

Reviewers
Update Map.jsx removing resetMapToCenter()

This PR simultaneously addresses #259 and #423

...by removing the resetMapToCenter() behavior from the app altogether, with the exception of the Home button.

This is a UI design decision targeting minimizing the amount of Moving Around as a whole.

+0 -4

1 comment

1 changed file

rudokemper

pr closed time in a month

issue closedTerrastories/terrastories

Sort stories dropdown resets map to default zoom/bounds when it shouldn't.

The sort stories dropdown should just filter the stories in the sidebar (in order A-Z, Z-A, or Most Recent). Currently, it seems to also triggering resetMapToCenter() as the map pans back to the default zoom and bounds. It should not be.

This behavior can be replicated in the demo: http://terrastories.herokuapp.com/en/home

  1. set the filters to Type of place > nonprofit campus. The map will zoom to the one point of this kind (as it should).
  2. use the Sort stories filter to anything else. It will zoom back to the default zoom/bounds (as it shouldn't).

image

closed time in a month

rudokemper

issue closedTerrastories/terrastories

[Map] Closing a marker popup should not reset the map zoom/bounds

This is an easy fix: when you close a popup, it should not reset the map view to home.

(If you're looking at a bunch of clustered markers close by each other, it's a little annoying to have to keep zooming in to examine the others.)

closed time in a month

rudokemper

Pull request review commentTerrastories/terrastories

Update to ReadMe Setup, Added Development section, fixed issue #446

+# Setup Windows++## Table of Contents++1. [Setup Docker](#setup-docker)++2. [Setup the application](#Setup-the-application)++3. [Run the application](#Run-the-application)++4. [Using the application](#using-the-application)++5. [Common setup errors](#having-troubles-check-our-common-errors--gotchas)++6. [Development](#development)++7. [Backup and restore the Terrastories database](#backup-and-restore-the-terrastories-database)++## Setup Docker++Docker is a platform that serves to deliver software in packages called containers. Terrastories makes use of this to deliver the +many facets included in the application in one easy package. ++Download Docker:++- https://hub.docker.com/editions/community/docker-ce-desktop-windows/++System prerequisites for Windows 10 Pro: +- 64 bit processor +- 4GB system RAM +- [virtualization must be enabled](https://mashtips.com/enable-virtualization-windows-10/). ++Setup instructions for Docker on Windows 10 Pro:++- https://docs.docker.com/docker-for-windows/install/++System prerequisites for Windows 10 Home: +- Ensure that you are running on version 2004 or higher, you will need to update your system if it is running on an older version+- Enable the WSL 2 feature on Windows, follow the instructions on this link to complete this step, this includes installing a Linux distribution such as Ubuntu+- To complete the previous steps, you must have a 64 bit processor, 4GB system RAM, and [virtualization must be enabled](https://mashtips.com/enable-virtualization-windows-10/). ++Setup instructions for Docker on Windows 10 Home:++- https://docs.docker.com/docker-for-windows/install-windows-home/++There is no need to import any repositories at this point through docker. +To verify that your docker is up and running, click on the app or check the menu bar. Docker should show that it is running: +<br>+<div style="text-align:center">+    <img src="dockerMac.png" alt="dockerImg" width="200"/>+</div>++## Setup the application+Create a fork of the Terrastories/terrastories repository. Now clone the repository locally to your computer. ++Using the source-code editor of your choice, open the terrastories repository. There, a file can be found called `.env.example`. +Copy the contents of this file into a newly created file called `.env` (Do not change .env.example!).++Now navigate to a site called [Mapbox](https://mapbox.com/signup) create an account, and copy the mapbox access token (either your default public token or a new one you create) found under your acccount. ++Navigate back to the `.env` file you created and replace where it says [your pk token here] with your mapbox access token. ++Open Powershell and navigate to the terrastories repository and run:++```+docker-compose build+```++This will download and build all the docker images used in this project. Upon completion you should see output similar to:++```+...+Successfully tagged terrastories:latest+```++## Run the application++Within Powershell, run:++```+docker-compose up+```++If this fails, make sure all firewalls are turned off and you have a secure connection to the internet. If it continues to fail, check +the [common setup errors](#having-troubles-check-our-common-errors--gotchas) section. ++This command should terminate with an output similar to:++```+terrastories_e2e_1 exited with code 0+```++Open another Powershell window and run the following command to setup:++```+docker-compose exec web bin/setup+```++This command runs a setup script that lives in bin/setup, which does:++- install ruby gems+- install javascript packages+- setup database+See the script file for the details.++Enter `localhost:3000` into your internet browser to view the application/++## Using the application ++Everytime you want to open and use the application, make sure you have docker desktop running and run the following command into+Powershell:++```+docker-compose up+```++You can view the running application at `localhost:3000`++It will take a moment to load when first opening the application +++## Having troubles? Check our common errors & gotchas++If you run into any problems getting the application to start, please check out a list of common errors & gotchas that we have put together [here](https://docs.google.com/document/d/1uSbQl56rAh3AA8Xm7IRZ8qepAMVN55ZOkAqQ8Kh423E/edit)!++Additionally, feel free to join us in Slack [here](https://t.co/kUtI3lnpW1) and find us in the channel #terrastories :) You can also post an issue and label it with `question`. We will get back to you ASAP!++## Development++Most developer contributions will be focused on the rails app. Because this project uses+docker, we already have a uniform ruby/rails development environment in our rails docker+image. Any time you need to run a rails command you should do so from a running docker+container to take advantage of this consistent environment. Use the following command to+open a bash console on the rails container:++```+docker-compose exec web /bin/bash+```++Now you can treat this console like any other development environment, running rails or+bundler commands as needed. **Please refrain from running such commands in your local+environment. Always use the rails container instead.**++Any changes to source files should be made directly in your local filesystem under the+`/opt/terrastories` directory using your preferred editing tools.++## Backup and restore the Terrastories database++Terrastories stores Places, Speakers, and Stories in a database (Postgres DB). it is possible to back these data up and restore them by running lines of code in a bash terminal. ++Using Powershell:++Backup the DB in PS with:++```+docker run --rm -v "terrastories_postgres_data:/pgdata" -v "$(pwd):/host" busybox tar -cvzf /host/db-backup-test.tgz -C /pgdata .+```++Restore a backup in PS with:++```+docker run --rm -i -v "terrastories_postgres_data:/pgdata" -v "$(pwd):/source/" busybox tar -xvzf /source/db-backup.tgz -C /pgdata+```++Note: the above code is assuming your build is called `terrastories`. It may be necessary to run `docker volume ls` to get the right Docker container name ending with `_postgres_data`.

I agree with @lauramosher's suggestion on this.

ohamuy

comment created time in a month

delete branch Terrastories/terrastories

delete branch : 446-translating-selected-phrase

delete time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha f11b9212bc975bc29099115ca74e46cbd460f00a

446 translating selected phrase (#460) * Update pt.yml, nl.yml, mat.yml, en.yml, es.yml, ja.yml to remove parentheses from `language_select` and provide translation for Portuguese, Dutch and Matawai. * Update application_helper.rb to add parentheses here.

view details

push time in a month

PR merged Terrastories/terrastories

Reviewers
446 translating selected phrase

This is a supplementary PR for https://github.com/Terrastories/terrastories/issues/446 that adds translation for selected in Matawai, Dutch, and Portuguese, and codes the parentheses into the helper file (rather than having them be in the i18n files).

+12 -12

1 comment

7 changed files

rudokemper

pr closed time in a month

issue closedTerrastories/terrastories

[CMS] [i18n] Language picker should show selected language, not default language.

Currently, the language picker (on the Welcome/Home) screen is showing a language as (default). Moreover, this (default) text is added independent of which language is actually selected as the default, in line 4 in the file application_helper.rb.

We should get rid of the (default) text altogether since that is not so interesting from a user point of view. Instead, we should have (selected) text showing for the language that is currently selected.

(As a follow up ticket, we can add the word selected to all of the i18n languages currently in Terrastories.)

closed time in a month

rudokemper

delete branch Terrastories/terrastories

delete branch : change-intropopup-color-to-white-2

delete time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha d46a768c314d443165210b039654191dbb981f70

Update introPopup.css changing h2 and h3 to white (#429)

view details

push time in a month

issue closedTerrastories/terrastories

Change h2 and h3 color in the intro popup back to white.

A previous PR changed a lot of text to #444 and inadvertently also changed the color of h2 and h3 in the intro popup. It should be changed back to white.

(Will submit PR momentarily)

closed time in a month

rudokemper

PR opened Terrastories/terrastories

Reviewers
446 translating selected phrase

This is a supplementary PR for https://github.com/Terrastories/terrastories/issues/446 that adds translation for selected in Matawai, Dutch, and Portuguese, and codes the parentheses into the helper file (rather than having them be in the i18n files).

+12 -12

0 comment

7 changed files

pr created time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 0a9e0cdf399626b1dea724a93326cf780098bd80

Update application_helper.rb

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 571d457ff49a4830924f07faea138f69f190b732

Update pt.yml

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 87de37325a8c59cb156a77a16750b7929897f8e9

Update nl.yml

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 9f3769aa0f6a2509fbf11931e0d0d4895ec49e05

Update ja.yml

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 508aa989a25921378900ac6ebd48029b8bfde9c8

Update es.yml

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 82041d509f0ff9eb2873204ec8eac46c5cdaa62d

Update en.yml

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha ccabb8fa23512ea2699ef45e47f814097e3e75ad

Update mat.yml

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 724453bbbef546e3d49b608c0e24555991640f2c

Update mat.yml

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha a7fbfdd5b9051a13530f561ca0942274a779df0d

Update nl.yml

view details

push time in a month

push eventTerrastories/terrastories

Rudo Kemper

commit sha 8968e59ea99b632cc9ac3fa75a21837371a437a0

Update pt.yml

view details

push time in a month

create barnchTerrastories/terrastories

branch : 446-translating-selected-phrase

created branch time in a month

GollumEvent
GollumEvent
GollumEvent
GollumEvent
GollumEvent
GollumEvent
GollumEvent
GollumEvent
GollumEvent

issue openedTerrastories/terrastories

[CMS] [i18n] Language picker should show selected language, not default language.

Currently, the language picker (on the Welcome/Home) screen is showing a language as (default). Moreover, this (default) text is added independent of which language is actually selected as the default, in line 4 in the file application_helper.rb.

We should get rid of the (default) text altogether since that is not so interesting from a user point of view. Instead, we should have (selected) text showing for the language that is currently selected.

(As a follow up ticket, we can add the word selected to all of the i18n languages currently in Terrastories.)

created time in 2 months

issue openedTerrastories/terrastories

[CMS] Prevent users from entering values outside of the threshold values for Place lat/long fields

Currently, it is possible to enter a value for the lat field that exceeds the boundaries of -90 to 90, and -180 to 190 for the long field.

This consequently breaks the map when you return to the home view.

An error message should pop up in Administrate when a user tries to enter values outside of the treshholds for the Latitude and Longitude fields.

created time in 2 months

issue openedTerrastories/terrastories

[CMS] Images uploaded to a story currently render as an audio file in the sidebar

Currently, it is possible to upload an image to a story. This is desirable behavior.

However, images are currently incorrectly rendering as an audio file in the sidebar. It should render as an image.

In the below screenshot, there is an image added for this story, not an audio file.

image

created time in 2 months

PR opened Terrastories/terrastories

Reviewers
Update SETUP.md

Added missing command for Windows for setup.

Addresses this issue: https://github.com/Terrastories/terrastories/issues/438

+5 -1

0 comment

1 changed file

pr created time in 2 months

push eventTerrastories/terrastories

Rudo Kemper

commit sha fe9cae9da2d6604693e211df0c08096aabf1e837

Update SETUP.md Added missing command for Windows for setup.

view details

push time in 2 months

issue openedTerrastories/terrastories

SETUP.md readme is missing one additional step for Windows

The command docker-compose exec web bin/setup does not work in Windows if you don't run the following command first: docker exec terrastories_web_1 /bin/bash.

created time in 2 months

more