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

Add 'spo field remove' command capability #738

Closed
9 tasks
Laul0 opened this issue Dec 29, 2018 · 8 comments
Closed
9 tasks

Add 'spo field remove' command capability #738

Laul0 opened this issue Dec 29, 2018 · 8 comments
Labels
good first issue new feature work in progress
Milestone
v1.15.0

Comments

@Laul0
Copy link
Contributor

Laul0 commented Dec 29, 2018 •
edited
Loading

Summary

Add a support for spo field remove command in order to :

  • Remove existing list column to an existing list
    • --fieldId / --fieldTitle (ID or Title not both)
    • --listId / --listTitle (ID or Title not both)
    • -u, --webUrl Web URL
    • --confirm (to suppress prompt)

To Do

  • Add command from solution
  • Add documentation of usage (MkDocs)
  • Validate cmdlet usage
  • Validate Unit test
@waldekmastykarz
Copy link
Member

Looks good! You want to take it yourself or let others pick it up?

@ktskumar
Copy link
Contributor

ktskumar commented Jan 4, 2019

May I work on this?

@waldekmastykarz
Copy link
Member

All yours @ktskumar 👏

@ktskumar
Copy link
Contributor

ktskumar commented Jan 7, 2019

Great. Thanks

@waldekmastykarz
Copy link
Member

Hey @ktskumar, are you still working on this?

@ktskumar
Copy link
Contributor

ktskumar commented Feb 6, 2019

Yes Waldek. Will pull the request this week.

@waldekmastykarz
Copy link
Member

Perfect! Looking forward to it!

@ktskumar
Copy link
Contributor

ktskumar commented Mar 5, 2019

Submitting pull request today

@ktskumar ktskumar mentioned this issue Mar 5, 2019
Closed
waldekmastykarz pushed a commit that referenced this issue Mar 10, 2019
@ktskumar @waldekmastykarz
Added the 'spo field remove' command solving #738
a94c27e
@waldekmastykarz waldekmastykarz added this to the v1.15.0 milestone Mar 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue new feature work in progress
Projects
None yet
Milestone
v1.15.0
Development

No branches or pull requests
3 participants
@waldekmastykarz @Laul0 @ktskumar