Skip to content

Club adif #2618

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
Merged
int2001 merged 9 commits into from
Dec 3, 2025
Merged

Club adif #2618

int2001 merged 9 commits into from
Dec 3, 2025

Conversation

@int2001
Copy link
Contributor

@int2001 int2001 commented Nov 30, 2025 •
edited
Loading

Introduce a new Role "Clubmember ADIF" (Clublevel 6) which is allowed to import ADIFS (for their own Call) and Export ADIFS (from their own Call / they made)

  • Introduce new role
  • Refactor ADIF-Views and Controller for more granular permission-management
  • Check Imported ADIF in case of Clubmember ADIF uploaded it. // If Clubmember ADIF uploads the OP-Call is strictly overwritten with the one from the Operator.
  • Export only QSOs from Operator
  • [-] Adjust LBA in similar way. // Will be done in extra-PR just for LBA by @AndreasK79

Testhints:

  • Edit Clubstation, set new permission ("Clubmember ADIF")
  • Test ADIF Functionality (everything below adif-menu) for normal user and admin.
  • Test ADIF Functionality (everything below adif-menu) for ClubOfficer on Clubstation.
  • Test ADIF Functionality (everything below adif-menu) for ClubMember on Clubstation.
  • Test ADIF Functionality (everything below adif-menu) for ClubMember ADIF on Clubstation

Expected outcome:

  • Normal User and Admin: As b4
  • ClubMember: as b4 - no export via ADIF-Menu (ADIF Menu-entry not even visible)
  • ClubMember ADIF: ADIF Menu visible.
    • Import: All ADIF Entries imported are strictly overwritten with Operators Call. No option to mark ADIF as already sent to 3rd party services, since OP != Owner of the call.
    • Export: Only QSOs the OP made will be exported. no "Mark exported QSOs as exported to LoTW"
  • Club Officer: As b4

@int2001 int2001 added enhancement New feature or request major impact Has a major impact on the code base new feature labels Nov 30, 2025
@int2001 int2001 marked this pull request as ready for review November 30, 2025 11:26
@int2001 int2001 removed the major impact Has a major impact on the code base label Dec 1, 2025
phl0
phl0 approved these changes Dec 3, 2025
View reviewed changes
Copy link
Contributor

@phl0 phl0 left a comment

Choose a reason for hiding this comment

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

Works for me. Only issue is that if I want a (new) user with export permissions I have to add him and then edit separately. If that is intended then idc. Otherwise please consider PR int2001#24. Wasn't able to add as suggestion because this area of the code is not in scope of this PR (yet).

@int2001 int2001 merged commit e6c2ff6 into wavelog:dev Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@phl0 phl0 phl0 approved these changes

@HB9HIL HB9HIL HB9HIL approved these changes

Assignees

No one assigned

Labels

enhancement New feature or request new feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@int2001 @phl0 @HB9HIL