Skip to content

Conversation

blip-lorist
Copy link
Contributor

Updated the "Add a Link" section for this page: http://docs.railsbridge.org/job-board/update_job_listings

At first glance, it seems like we're supposed to be updating the edit view, but it's really the index view. Tried to highlight by calling out the specific file path.

Our users probably aren't going to know they can hit `/jobs/:id/edit` to visit the edit form, so let's add a link to it on the jobs index so we end up with this (we're just adding the line with the `<h5>` header in it ... don't copy and paste the whole thing!):
Our users probably aren't going to know they can hit `/jobs/:id/edit` to visit the edit form, so let's add a link so it's easy to find!

In `app/views/jobs/index.html.erb`, just add this line with the `<h5>` header in it ... don't copy and paste the whole thing!
Copy link
Member

Choose a reason for hiding this comment

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

Hi @lorainekv! This line needs to be indented the same as the rest of the block of markdown, or else the markdown stops getting parsed correctly:

screen shot 2016-07-01 at 7 45 31 pm

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gotcha, updated @lilliealbert

@lilliealbert
Copy link
Member

Awesome, can you squash the two commits so it's like the indentation thing never happened? ✨

@blip-lorist
Copy link
Contributor Author

Bwahahah I think I made it worse. Not familiar with rebasing for squashes.

@lilliealbert
Copy link
Member

hee hee. 😄

here's a guide! http://railsbridge.github.io/bridge_troll/

@blip-lorist blip-lorist force-pushed the documentation-updates branch from 91c152f to f695789 Compare July 2, 2016 05:53
@blip-lorist
Copy link
Contributor Author

Ah discovered what I was doing wrong - was running git pull before my git push - f Mahalo for sending that doc!

@lilliealbert
Copy link
Member

Looks good! Thanks for the update! ✨

@lilliealbert lilliealbert merged commit 804a73f into railsbridge:master Jul 2, 2016
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.

2 participants