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

9658 api #28

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open

9658 api #28

wants to merge 34 commits into from

Conversation

withanage
Copy link
Owner

No description provided.

@ipula ipula force-pushed the 9658-api branch 5 times, most recently from 54011bb to 455963f Compare September 10, 2024 09:05
@ipula ipula force-pushed the 9658-api branch 8 times, most recently from a1e6b9f to 6f73d72 Compare September 17, 2024 10:40
withanage

This comment was marked as duplicate.

Copy link
Owner Author

Choose a reason for hiding this comment

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

@ipula PKP license header fails here.

Copy link

Choose a reason for hiding this comment

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

done

Copy link
Owner Author

@withanage withanage left a comment

Choose a reason for hiding this comment

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

@ipula PKP license header fails here.

@ipula
Copy link

ipula commented Sep 24, 2024

requested changes are done

@ipula ipula force-pushed the 9658-api branch 5 times, most recently from 3ef084a to 59efbc9 Compare September 25, 2024 20:32
defstat and others added 25 commits September 26, 2024 11:39
* pkp#10188 Convert Notes to an Eloquent model

* pkp#10188 review feedback for converting Notes to Eloquent

* pkp#10188 further review feedback for converting Notes to Eloquent
@ipula ipula force-pushed the 9658-api branch 3 times, most recently from f3a84b0 to f9513e0 Compare September 30, 2024 17:16
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.

4 participants