Skip to content

fix typo, probably introduced by v1 code - #3325

Merged
martaribeiro merged 3 commits into
DMPRoadmap:developmentfrom
nicolasfranck:main
Oct 16, 2023
Merged

fix typo, probably introduced by v1 code#3325
martaribeiro merged 3 commits into
DMPRoadmap:developmentfrom
nicolasfranck:main

Conversation

@nicolasfranck

@nicolasfranck nicolasfranck commented Jul 13, 2023

Copy link
Copy Markdown
Contributor

fix typo, probably introduced by v1 code

@per_page attribute does not exists here, so returns an internal server error containing

[6689aad6-1f63-4bfc-aae0-3f331d464dc1] NoMethodError (undefined method `>' for nil:NilClass):
[6689aad6-1f63-4bfc-aae0-3f331d464dc1]   
[6689aad6-1f63-4bfc-aae0-3f331d464dc1] app/controllers/api/v0/plans_controller.rb:104:in `index'
@nicolasfranck nicolasfranck mentioned this pull request Jul 13, 2023
Open
@nicolasfranck nicolasfranck mentioned this pull request Sep 4, 2023
Closed
nicolasfranck added a commit to DMPbelgium/roadmap that referenced this pull request Sep 21, 2023
@nicolasfranck
prematurely fix bug as fixed by DMPRoadmap/roadmap#3325
795a2de
@don-stuckey
don-stuckey changed the base branch from main to development October 2, 2023 15:53
@martaribeiro
martaribeiro merged commit 83ad549 into DMPRoadmap:development Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@nicolasfranck @martaribeiro