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

Assert tkts option defaults should include everything unless specified otherwise #79

Open
jwflory opened this issue Nov 23, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@jwflory
Copy link
Member

jwflory commented Nov 23, 2023

Summary

When a user imports tickets from Pagure to GitLab, the tool should default to including everything unless specified otherwise.

Background

When a user is running the tool, they are likely wanting to make a full copy of what is in Pagure to the extent as much is possible. Different metadata matters to different people. Things like labels and the current status of tickets could be very important data points for a team. Of course, not everyone reads the documentation and they might not realize what all options are available.

I predict it would result in a better user experience if the tool defaults to importing everything, to avoid a user having a negative user experience of realizing that data they thought would come with them did not.

Details

This is a "good first issue" that a new contributor could help with.

We need to change the default options in main.py so that the flags are enabled by default. Perhaps the names and help messages of the flags themselves could be made more clear as well about being omissions instead of additions when invoked.

https://github.com/fedora-infra/pagure-exporter/blob/3d77bfe1a461e7daab6a89b3affa2c2a88a03887/pagure_exporter/main.py#L120-L151

Outcome

Better user experience when using the tool by porting out all relevant metadata and all issues (FULL) by default.

@gridhead gridhead added the enhancement New feature or request label Nov 23, 2023
@gridhead gridhead assigned gridhead and unassigned gridhead Dec 11, 2023
@gridhead gridhead added the good first issue Good for newcomers label Mar 14, 2024
@brngylni
Copy link
Contributor

I'd be happy to work on this one too. So you can assign it to me.

@gridhead gridhead linked a pull request Mar 19, 2024 that will close this issue
@gridhead
Copy link
Member

@brngylni I have assigned you this issue ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants