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

rebased by standing at my rails-4.2 branch #266

Merged
merged 169 commits into from
Mar 20, 2018

Conversation

Souravirus
Copy link

Make sure these boxes are checked before your pull request is ready to be reviewed and merged. Thanks!

  • all tests pass -- rake test:all
  • code is in uniquely-named feature branch, and has been rebased on top of latest master (especially if you've been asked to make additional changes)
  • pull request is descriptively named with #number reference back to original issue

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/wiki/contributing-to-public-lab-software

We have a loose schedule of reviewing and pulling in changes every Tuesday and Friday, and publishing changes on Fridays. Please alert developers on [email protected] when your request is ready or if you need assistance.

Thanks!

jywarren and others added 30 commits January 25, 2018 20:57
* Lock paperclip to required version

* Exclude critical files

* Disable magic comment

* Safe navigate on params

* Move Style/FrozenStringLiteralComment to .rubocop.yml

* Rubocop autofix
* form http request changes to post

* Tests for long answer using POST request

* Commented test for GET request

* Uncommented the test for large answers
…bliclab#2109)

* Fixes #publiclab#2079

Fixes publiclab#2079
This commit adds active user tab to publiclab.org/spam.

* Add people tab to /spam page for moderators(Fixes publiclab#2079)
* Added links to the profile page

* Stats display
* Update _sidebar.html.erb

* Update I18n_test.rb
* Update typeahead_service.rb

* Update typeahead_service.rb

* Update restful_typeahead.js

* Update revision.rb
* chg: added button to join the powertag group's tag

* add: added check to see if current_user is available before showing the button

* fix: added defined? to check if the curre_user variable is defined

* fix: added defined? check to see if the current_user variable is defined

* chg: modified redirection for the create action

* chg: added button for when the user is not currently logged in

* add: added current_user as a parameter

* fix: added defined? to check if the curre_user variable is defined

* chg: added button for when the user is not currently logged in

* text tweaks

* tweak

* Update user_tags_controller_test.rb

* Update wiki_controller_test.rb

* Update node_shared.rb

* Update application_helper.rb

* Hopefully fix test failures

* Update application_helper.rb
)

* Added maps

* API made for people maps

* Added API url for People map

* Made helper function for People Map API

* Updated self.people_map

* Created _peopleLeaflet.html.erb

* Drupal_user removed

* Indentation done

* search Recent People functionality

* search Recent People helper
…ab#2078)

* Refactor code for user's social links

* Fix Relationship tests

* Add validations to Relationship model
)

*  Make tags visible to other users (fixes publiclab#2138)

 publiclab#2138

 The profile tags are currently only visible to the user themselves or
 to admins. The tags should be visible to everyone, while the "add tags" input should maintain its current visibility status (owner or administrator).

* Add a parent param to the tag/tagging partial.

This cleverly hides the input but only if it is rendered inside the
profile view.

* Improve the condition

* If parent is unset, set parent to nil
* Add ruby-2.3.0 alert to README.md

* made it as a small note

* change from 2.3.0 to 2.3.4

* small change

* Changed bundle install statement
* Add "comment" and "barnstar" headings to profile nav

fixes publiclab#2165

* minor changes
* User must be logged out to access /login page

* Update user_sessions_controller.rb
* Some modification to user dropdown menu on navbar

* Added tests
@Souravirus
Copy link
Author

Hey I did as you told by standing at jywarren:rails-4.2 branch I rebased it to publiclab:master branch. Now is it OK @jywarren

@Souravirus Souravirus mentioned this pull request Mar 14, 2018
3 tasks
@jywarren
Copy link
Owner

Ah, i think this is good, so now this is where you can finish the 4.2 port, right? @Gauravano how does this look to you?

@Souravirus
Copy link
Author

Yeah @jywarren I will do rails 4.2 port on it.

@grvsachdeva
Copy link

hi @jywarren @Souravirus , looks good to me

@Souravirus
Copy link
Author

Souravirus commented Mar 20, 2018

Hey @jywarren, are we ready to merge this PR. So, that I can start updating to rails 4.2 for new commits.

@jywarren jywarren merged commit 6aa3149 into jywarren:rails-4.2 Mar 20, 2018
@jywarren
Copy link
Owner

OK!

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.