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.

Sign up for GitHub

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

Jump to bottom

The old null author affiliation are also migrated #10905

Closed
bozana opened this issue Feb 7, 2025 · 1 comment
Closed

The old null author affiliation are also migrated #10905

bozana opened this issue Feb 7, 2025 · 1 comment
Assignees
@bozana
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Milestone
3.5 Internal

Comments

@bozana
Copy link
Collaborator

bozana commented Feb 7, 2025 •
edited
Loading

Earlier in author_settings table we had also setting_value = null for setting_name = 'affiliation'. Those should not be migrated, because those authors have no affiliation.

PRs:
@bozana bozana added this to the 3.5 Internal milestone Feb 7, 2025
@bozana bozana self-assigned this Feb 7, 2025
@bozana bozana added the Bug:1:Low A bug that does not have a severe consequence or affects a small number of users. label Feb 7, 2025
@bozana bozana mentioned this issue Feb 7, 2025
Merged
@bozana bozana closed this as completed Feb 7, 2025
GaziYucel added a commit to GaziYucel/pkp-lib that referenced this issue Feb 8, 2025
@GaziYucel
pkp#10905 do not migrate null author affiliation
4efde9d
@GaziYucel GaziYucel mentioned this issue Feb 8, 2025
Merged
@GaziYucel
Copy link
Contributor

We need this also for ROR affiliations.

@GaziYucel GaziYucel mentioned this issue Feb 8, 2025
Open
GaziYucel added a commit to GaziYucel/pkp-lib that referenced this issue Feb 8, 2025
@GaziYucel
pkp#10905 do not migrate null author affiliation
3f8bb55
GaziYucel added a commit to GaziYucel/pkp-lib that referenced this issue Feb 9, 2025
@GaziYucel
pkp#10905 validate language code 'no_lang_code' correctly
73615a8
bozana added a commit that referenced this issue Feb 9, 2025
@bozana
Merge pull request #10912 from GaziYucel/i10905
fe8d119 
#10905 do not migrate null author affiliation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bozana bozana

Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Projects
None yet
Milestone
3.5 Internal
Development

No branches or pull requests
2 participants
@bozana @GaziYucel