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

1864 changelog backend [WIP] #2127

Closed
wants to merge 69 commits into from
Closed

Conversation

kinstella
Copy link
Contributor

This pull request makes the following changes:

  • Adds a posts_changelog table
  • Intercepts updates to Post fields, writes content to a table
  • Provides API for manually writing comments to the changelog
  • Provides route for viewing changelog of each Post
  • Restricts changelog access via the API

Test checklist:

  • [ ]

Fixes ushahidi/platform# .

Ping @ushahidi/platform

* @license https://www.gnu.org/licenses/agpl-3.0.html GNU Affero General Public License Version 3 (AGPL3)
*/

use Ushahidi\Core\Exception\FormatterException;
Copy link
Contributor

Choose a reason for hiding this comment

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

spaces are off I think

$this->changelog_repo = $changelog_repo;
}

//TODO: this is ugly, because we're only receiving the ID here.
Copy link
Contributor

Choose a reason for hiding this comment

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

Indent

}catch (Exception $e)
{
Kohana::$log->add(Log::INFO, 'trying to handle a collection change event.' );
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Indenting I think I wrong here too

@willdoran
Copy link
Contributor

@kinstelli I think this is still paused until develop is stable?

@kinstella kinstella changed the title [WIP] 1864 redo changelog like tags 1864 redo changelog like tags [WIP] Nov 9, 2017
@kinstella kinstella changed the title 1864 redo changelog like tags [WIP] 1864 changelog backend [WIP] Nov 23, 2017
@rowasc rowasc added this to the Cleanup old PRs milestone Sep 19, 2018
@rowasc rowasc mentioned this pull request Sep 2, 2019
@rowasc
Copy link
Contributor

rowasc commented Sep 2, 2019

Closing per agreement that stale PRs won't remain in the list of open PRs. The branch will still be around and the ticket #1864 will remain open when we want to pick it up .

@rowasc rowasc closed this Sep 2, 2019
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

5 participants